-
Notifications
You must be signed in to change notification settings - Fork 283
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
V3.7.x.updates #5424
V3.7.x.updates #5424
Conversation
Hi @HGWright This step is almost entirely about getting the docs to look right. |
Sorry, please hold off. Fixing a docs-build problem ... |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## v3.7.x #5424 +/- ##
=======================================
Coverage 89.37% 89.37%
=======================================
Files 89 89
Lines 22443 22443
Branches 5381 5381
=======================================
Hits 20058 20058
Misses 1639 1639
Partials 746 746 ☔ View full report in Codecov by Sentry. |
Ok fixed ! Please review @HGWright ! |
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.
Good to see RTD working on this PR. I think that there are 2 option with regards to the 'Major Highlights' of the release, and I will leave the decision to you @pp-mo.
- Leave it as is, because there is nothing of the size that usually constitutes a major highlight.
- Take the oppertunity to highlight some more niche but still good/important changes.
Ok thanks @HGWright |
Looks better! Very happy for this to be merged. |
Great, thanks. |
Whatsnew text finalisation + renames, suitable for release branch docs