-
Notifications
You must be signed in to change notification settings - Fork 80
Prepare for the release (changelog, README, copyright, etc) #221
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #221 +/- ##
=======================================
Coverage 92.29% 92.29%
=======================================
Files 25 25
Lines 1714 1714
Branches 201 201
=======================================
Hits 1582 1582
Misses 110 110
Partials 22 22
Continue to review full report at Codecov.
|
@iainrussell I'll consider this PR uncontroversial enough to merge it without your explicit approval, but I need you have a look at #220 as it is more tricky and I'll need your approval to merge and proceed with the release. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #221 +/- ##
==========================================
+ Coverage 92.22% 92.24% +0.02%
==========================================
Files 25 25
Lines 1723 1754 +31
Branches 202 206 +4
==========================================
+ Hits 1589 1618 +29
- Misses 111 112 +1
- Partials 23 24 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Add changelog entries for several older issues:
import cfgrib
if ecCodes is not present on the system. #14cfgrib
#50