Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Improve handling around required flags #10

Merged
merged 1 commit into from
Jan 18, 2022
Merged

Improve handling around required flags #10

merged 1 commit into from
Jan 18, 2022

Conversation

ccremer
Copy link
Contributor

@ccremer ccremer commented Jan 17, 2022

Summary

  • If a required flag was not given, it would show usage without displaying the error
  • It turns out determining whether to print Metadata based on the arguments is nigh impossible. So let's leave that up to the subcommands

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Update tests.

@ccremer ccremer merged commit 58ffc5c into master Jan 18, 2022
@ccremer ccremer deleted the flags branch January 18, 2022 12:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants