-
Notifications
You must be signed in to change notification settings - Fork 129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow selection of metadata ID column #1240
Conversation
Since it is not a positional argument.
Previously, this flag to customize the default value was only available in augur filter. Add it to other subcommands to parallel the existing support for --metadata-valid-delimiters which serves a similar purpose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re: @jameshadfield in original issue description:
We don't need to add it to
export v1
.
I opted to add this to export v1
to keep things consistent, plus it's an easy change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1240 +/- ##
==========================================
+ Coverage 68.88% 68.90% +0.02%
==========================================
Files 64 64
Lines 6939 6944 +5
Branches 1693 1693
==========================================
+ Hits 4780 4785 +5
Misses 1854 1854
Partials 305 305
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for working on this! LGTM by inspection.
Description of proposed changes
See commit messages.
Related issue(s)
Resolves #1237.
Testing
No tests added.
Checklist