Skip to content
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

Move the NCBI field map to config #219

Merged
merged 1 commit into from
Nov 2, 2023
Merged

Move the NCBI field map to config #219

merged 1 commit into from
Nov 2, 2023

Conversation

j23414
Copy link
Contributor

@j23414 j23414 commented Oct 31, 2023

Description of proposed changes

To be consistent with the handling of nextclade-field-map.tsv, move ncbi-field-map.tsv to config.
This change is motivated by the following comment:

Related issue(s)

Checklist

  • Checks pass

@j23414 j23414 requested a review from a team October 31, 2023 20:21
@j23414 j23414 mentioned this pull request Oct 31, 2023
2 tasks
To be consistent with the nextclade-field-map.tsv, move the ncbi-field-map.tsv
to config.
@j23414 j23414 force-pushed the ncbi_field_map_config branch from 66a45df to f2f7a61 Compare November 1, 2023 23:54
@j23414 j23414 merged commit 59d3f4b into master Nov 2, 2023
@j23414 j23414 deleted the ncbi_field_map_config branch November 2, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants