search

itzg / rcon-cli

A little RCON cli based on james4k's RCON library for golang
Apache License 2.0
181 stars 17 forks source link

please conform to the newer ~/.config/rcon-cli dotfile standard #5

Open cyclingwithelephants opened 4 years ago

cyclingwithelephants commented 4 years ago

I'd be happy to submit a PR making this change as long as you'll give me some acceptance criteria :)

It could be made backwards compatible by falling back to ~/.rcon-cli.yaml for example

itzg commented 4 years ago

If you mean adjusting the viper config

https://github.com/itzg/rcon-cli/blob/ef0727479c9ccf37ffdef2ed929e0e51402c97da/cmd/root.go#L90

to be consistent with that standard, then I’d say go for it! :)

In my primary use cases for rcon-cli I pass config via env vars, so my only acceptance criteria is keep the code change minimal.