Skip to content
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

Enable version warning banners for docs #7139

Merged
merged 4 commits into from Sep 22, 2023

Conversation

lagru
Copy link
Member

@lagru lagru commented Sep 16, 2023

Description

If the doc version isn't the preferred one, a warning banner is shown at the top. This also accounts for dev builds.

Blocked until pydata/pydata-sphinx-theme#1450 pydata/pydata-sphinx-theme#1446 is resolved and merged.

Partially addresses #7138.

Checklist

For reviewers: see #7139 (comment)

Release note

Summarize the introduced changes in the code block below in one or a few sentences. The
summary will be included in the next release notes automatically:

...

If the doc version isn't the preferred one, a warning banner is shown
at the top. This also accounts for dev builds.
@lagru lagru added the 📄 type: Documentation Updates, fixes and additions to documentation label Sep 16, 2023
@lagru lagru added the ⬆️ Upstream Needs help from or involves an upstream project label Sep 16, 2023
0.14.1 fixes a broken version comparison in the javascript of the
version warning banner. See
pydata/pydata-sphinx-theme#1446 for more.
@pep8speaks
Copy link

pep8speaks commented Sep 22, 2023

Hello @lagru! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-09-22 10:03:12 UTC

@lagru lagru marked this pull request as ready for review September 22, 2023 10:04
@lagru
Copy link
Member Author

lagru commented Sep 22, 2023

Should be ready to review / merge.

For reviewers: the updated doc/source/_static/version_switcher.json with the new setting "preferred": true will only be available once this PR is merged. However you can test this PR locally by replacing the URL to version_switcher.json with this one: https://gist.githubusercontent.com/lagru/8bce79c510b9026dab455c0a96a2c0b3/raw/d7d6159090de17cf54b55b457b198284c4078da0/version_switcher.json

{
"name": "PyPI",
"url": "https://pypi.org/project/scikit-image/",
"icon": "fa-solid fa-box",
},
],
"navbar_align": "left",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This and the added / removed configuration in "icon_links" and "github_url" accounts for some updates between 0.13 and 0.14.

@jarrodmillman jarrodmillman merged commit 27cde7c into scikit-image:main Sep 22, 2023
20 checks passed
@stefanv stefanv added this to the 0.22 milestone Sep 22, 2023
@lagru lagru removed the ⬆️ Upstream Needs help from or involves an upstream project label Sep 23, 2023
@soupault soupault mentioned this pull request Sep 16, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📄 type: Documentation Updates, fixes and additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants