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

sitemap.py: sort work list for a stable sitemap.xml #4615

Merged
merged 1 commit into from
Aug 25, 2023

Conversation

elliefm
Copy link
Contributor

@elliefm elliefm commented Aug 23, 2023

The default ordering of env.found_docs seems to be unstable in newer Sphinx versions, resulting in sitemap.xml changes every single time the website builder runs, even if nothing actually changed.

With this list now processed in sorted order, sitemap.xml shouldn't change unless we've actually added, removed, or renamed documents.

The default ordering of env.found_docs seems to be unstable in newer
Sphinx versions, resulting in sitemap.xml changes every single time
the website builder runs, even if nothing actually changed.

With this list now processed in sorted order, sitemap.xml shouldn't
change unless we've actually added, removed, or renamed documents.
@elliefm elliefm added backport-to-3.6 for PRs that are to be backported to 3.6 backport-to-3.4 for PRs that are to be backported to 3.4 backport-to-3.2 for PRs that are to be backported to 3.2 backport-to-3.8 for PRs that are to be backported to 3.8 labels Aug 23, 2023
@elliefm
Copy link
Contributor Author

elliefm commented Aug 23, 2023

@elliefm: this needs to be backported all the way to 2.5

@elliefm elliefm merged commit 6c852ba into cyrusimap:master Aug 25, 2023
1 check passed
@elliefm elliefm removed backport-to-3.6 for PRs that are to be backported to 3.6 backport-to-3.8 for PRs that are to be backported to 3.8 backport-to-3.4 for PRs that are to be backported to 3.4 backport-to-3.2 for PRs that are to be backported to 3.2 labels Aug 25, 2023
@elliefm
Copy link
Contributor Author

elliefm commented Aug 25, 2023

This is now backported as far as 3.0. Turns out 2.5 doesn't have the sitemap builder, so doesn't need the fix.

@elliefm elliefm deleted the v39/sort-sitemap-work-list branch August 25, 2023 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants