-
Notifications
You must be signed in to change notification settings - Fork 35
release/1.8.0 #1734
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
release/1.8.0 #1734
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1734 +/- ##
==========================================
- Coverage 95.36% 93.09% -2.28%
==========================================
Files 189 189
Lines 16536 16536
==========================================
- Hits 15770 15394 -376
- Misses 766 1142 +376
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
CI running on |
mikemhenry
left a comment
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.
LGTM! Just one nit on the "sentence per line" rule for rst files
Co-authored-by: Mike Henry <11765982+mikemhenry@users.noreply.github.com>
|
No API break detected ✅ |
release prep for v1.8.0
Make the PR:
release-prep-v1.2.0. Note: please follow semantic versioning.news/rever.rstfiles. You can backfill any additional items by making a new .rst, e.g.backfill.rstrever 1.2.0. This will auto-commitdocs/CHANGELOG.mdand remove the.rstfiles fromnews/.docs/CHANGELOG.rstlooks correct.main.showcase/openfe_showcase.ipynband.binder/environment.ymlAfter Merging the PR follow this guide