Skip to content

Conversation

NeckBeardPrince
Copy link
Contributor

Changed extension from YAML to YML and added more examples to config.yml.

@destari
Copy link
Contributor

destari commented Sep 11, 2025

These changes look pretty good in general, and the formatting tweaks are great!

One thing, to make it all consistent, we would also need to change config.go line 175, from yaml to yml.

@destari destari self-requested a review September 11, 2025 15:25
@NeckBeardPrince
Copy link
Contributor Author

NeckBeardPrince commented Sep 11, 2025

These changes look pretty good in general, and the formatting tweaks are great!

One thing, to make it all consistent, we would also need to change config.go line 175, from yaml to yml.

It looks like Gonzo picks up the config file regardless of whether it's yml or yaml. I just tried both, and Gonzo picks it up.

In main.go the default is config.yml and on line 175 it's looking for the ConfigType("yaml"). I think ConfigType in the viper module looks for all the different extensions of YAML.

❯ gonzo
2025/09/11 10:38:42 Using config file: /Users/user/.config/gonzo/config.yaml
❯ gonzo
2025/09/11 10:39:12 Using config file: /Users/user/.config/gonzo/config.yml

@destari destari merged commit 88b659d into control-theory:main Sep 11, 2025
@NeckBeardPrince NeckBeardPrince deleted the fix/fix_examples_link branch September 11, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants