-
-
Notifications
You must be signed in to change notification settings - Fork 53
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
Rename eccode to ec #1413
base: main
Are you sure you want to change the base?
Rename eccode to ec #1413
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1413 +/- ##
==========================================
+ Coverage 42.51% 46.80% +4.29%
==========================================
Files 117 118 +1
Lines 8327 8332 +5
Branches 1963 1354 -609
==========================================
+ Hits 3540 3900 +360
+ Misses 4582 4239 -343
+ Partials 205 193 -12 ☔ 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.
Looks great, thanks!
Oh actually, just noticed one more small thing, we might want to update this sentence: bioregistry/docs/CONTRIBUTING.md Line 73 in 7fae7ba
|
This pull request addresses the following issue: #1404
The following changes are implemented:
eccode
prefix toec
preferred_prefix
is set toEC
eccode
as a synonymprovides: eccode
toprovides: ec
A few additional files that referenced
eccode
are updated to reflect this change as well.