mirror of
https://github.com/taigrr/wtf
synced 2025-01-18 04:03:14 -08:00
Decouple modules from global config
Rather than referencing wtc.Config, instead pass the global config Also, look up config for the module early and pass that in sooner, to deal with fewer long paths and get rid of the ConfigKey variable
This commit is contained in:
@@ -18,35 +18,34 @@ type Settings struct {
|
||||
scheduleIdentifierType string
|
||||
}
|
||||
|
||||
func NewSettingsFromYAML(name string, ymlConfig *config.Config) *Settings {
|
||||
localConfig, _ := ymlConfig.Get("wtf.mods." + configKey)
|
||||
func NewSettingsFromYAML(name string, ymlConfig *config.Config, globalConfig *config.Config) *Settings {
|
||||
|
||||
settings := Settings{
|
||||
common: cfg.NewCommonSettingsFromYAML(name, configKey, ymlConfig),
|
||||
common: cfg.NewCommonSettingsFromModule(name, ymlConfig, globalConfig),
|
||||
|
||||
apiKey: localConfig.UString("apiKey", os.Getenv("WTF_OPS_GENIE_API_KEY")),
|
||||
displayEmpty: localConfig.UBool("displayEmpty", true),
|
||||
scheduleIdentifierType: localConfig.UString("scheduleIdentifierType", "id"),
|
||||
apiKey: ymlConfig.UString("apiKey", os.Getenv("WTF_OPS_GENIE_API_KEY")),
|
||||
displayEmpty: ymlConfig.UBool("displayEmpty", true),
|
||||
scheduleIdentifierType: ymlConfig.UString("scheduleIdentifierType", "id"),
|
||||
}
|
||||
|
||||
settings.schedule = settings.arrayifySchedules(localConfig)
|
||||
settings.schedule = settings.arrayifySchedules(ymlConfig, globalConfig)
|
||||
|
||||
return &settings
|
||||
}
|
||||
|
||||
// arrayifySchedules figures out if we're dealing with a single project or an array of projects
|
||||
func (settings *Settings) arrayifySchedules(localConfig *config.Config) []string {
|
||||
func (settings *Settings) arrayifySchedules(ymlConfig *config.Config, globalConfig *config.Config) []string {
|
||||
schedules := []string{}
|
||||
|
||||
// Single schedule
|
||||
schedule, err := localConfig.String("schedule")
|
||||
schedule, err := ymlConfig.String("schedule")
|
||||
if err == nil {
|
||||
schedules = append(schedules, schedule)
|
||||
return schedules
|
||||
}
|
||||
|
||||
// Array of schedules
|
||||
scheduleList := localConfig.UList("schedule")
|
||||
scheduleList := ymlConfig.UList("schedule")
|
||||
for _, scheduleName := range scheduleList {
|
||||
if schedule, ok := scheduleName.(string); ok {
|
||||
schedules = append(schedules, schedule)
|
||||
|
||||
Reference in New Issue
Block a user