-
Notifications
You must be signed in to change notification settings - Fork 603
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
Add a Release News section to the table of contents #5548
Add a Release News section to the table of contents #5548
Conversation
…o newopmathintro
Co-authored-by: Thomas R. Bromley <49409390+trbromley@users.noreply.github.com>
…lane into newopmathintro
…o newopmathintro
…o newopmathintro
…o newopmathintro
Co-authored-by: Thomas R. Bromley <49409390+trbromley@users.noreply.github.com>
…o newopmathintro
development/release_notes.md | ||
development/deprecations |
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.
I decided not to move the location of these two pages in the codebase, because this location also determines the URL and we have shared these pages extensively over the year.
doc/introduction/returns.rst
Outdated
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.
This page has been around for ~12 months and the toggle was even removed in 0.33. Now seems a good time to remove this page.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## newopmathintro #5548 +/- ##
=================================================
Coverage ? 99.66%
=================================================
Files ? 406
Lines ? 37635
Branches ? 0
=================================================
Hits ? 37510
Misses ? 125
Partials ? 0 ☔ View full report in Codecov by Sentry. |
Closed in favour of #5563. |
Context:
#5483 adds a new quickstart guiding users through the changes to op arithmetic. However, there are too many quickstarts in the table of contents.
Description of the Change:
Benefits:
A clearer distinction between quickstarts (guiding users to quickly get started with the library) and news/time-based pages (now called "Release news").
Closed in favour of #5563.