-
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
Port ingest/transform-strain-names
to augur curate transform-strain-name
[#1486]
#1514
Conversation
@joverlee521 I wonder if |
Yeah, I was debating on this when I wrote up the issue. I think eventually the command would be expanded to transform strain names as described in #1515. |
Yeah, okay, fair. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1514 +/- ##
==========================================
+ Coverage 69.54% 69.63% +0.08%
==========================================
Files 73 74 +1
Lines 7786 7812 +26
Branches 1904 1910 +6
==========================================
+ Hits 5415 5440 +25
Misses 2086 2086
- Partials 285 286 +1 ☔ View full report in Codecov by Sentry. |
* Convert script over to expected sub-cmd style * Add type hints throughout * Add tests
455a76b
to
d33556c
Compare
Description of proposed changes
Port
ingest/transform-strain-names
toaugur curate transform-strain-name
, add tests.Related issue(s)
#1486
nextstrain/ingest#43
Checklist