drsybren/Flamenco
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Manager: variable replacement, remove warning when no variables exist

Browse Source 
It's perfectly fine to run Flamenco without any variables, so do not log
a warning when this situation occurs.
This commit is contained in:
Sybren A. Stüvel 2024-06-25 11:19:59 +02:00
parent 374a8c0a6f
commit 5249744d0a
1 changed files with 0 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
internal/manager/config/variables.go
View File

@ -39,15 +39,6 @@ func (c *Conf) NewVariableToValueConverter(audience VariableAudience, platform V
// NewVariableExpander returns a new VariableExpander for the given audience & platform. // NewVariableExpander returns a new VariableExpander for the given audience & platform.
func (c *Conf) NewVariableExpander(audience VariableAudience, platform VariablePlatform) *VariableExpander { func (c *Conf) NewVariableExpander(audience VariableAudience, platform VariablePlatform) *VariableExpander {
// Get the variables for the given audience & platform.
varsForPlatform := c.getVariables(audience, platform)
if len(varsForPlatform) == 0 {
log.Warn().
Str("audience", string(audience)).
Str("platform", string(platform)).
Msg("no variables defined for this platform given this audience")
}
return &VariableExpander{ return &VariableExpander{
oneWayVars: varsForPlatform, oneWayVars: varsForPlatform,
managerTwoWayVars: c.GetTwoWayVariables(audience, c.currentGOOS), managerTwoWayVars: c.GetTwoWayVariables(audience, c.currentGOOS),