Allow the default refresh.yml file to exist in multiple locations

syntaqx commented 5 years ago

Rather than limit the configuration to just refresh.yml, this allows for developers to use their preferred yaml file extension, and allows for the dotfile prefix convention that is implemented by other tooling.

When a config, c flag is explicitly set, only that file is looked for.
Don't exit the run action if no configuration is found, the defaults are sane.
Tell the configuration file that ends up being used.

There's also a few minor err != nil cleanups I added while I was in the file, nothing logic changing.

Unrelated note: I've opened a PR for goreleaser/godownloader so the project can benefit from a shell installer. Thought you'd like to know!

markbates commented 5 years ago

Awesome! once the tests run I'll merge and release. Thanks.

hlubek commented 4 years ago

I think this commit broke the init command: cfgFile will be empty and Dump cannot open an empty file for writing (the error is not handled in Run though. See #36.

markbates / refresh

Allow the default refresh.yml file to exist in multiple locations #28