search

hawkins / watchdog

⚠️ Watch filesystem for changes and then run a command
2 stars 0 forks source link

Require glob or path at Clap level #14

Closed palfrey closed 5 years ago

palfrey commented 5 years ago

It fails later on if neither are set, but can be caught at the arg parsing level with Arg::required_unless_one which would be slightly nicer than a panic. Otherwise, a good tool, thanks.

hawkins commented 5 years ago

Nice catch - I'm not very familiar with Clap, so I wasn't aware of that feature. Thanks for both using the tool and reporting to help me improve it! :smile:

Edit: happy to accept a PR if you'd like, or else I'll get this in the next release myself

hawkins commented 5 years ago

Done in v0.2.6 . Thank you very much! 🎉 https://github.com/hawkins/watchdog/releases/tag/v0.2.6