8000 Fix: Decrase anchor validation treeprocessor prioriy by joapuiib · Pull Request #3945 · mkdocs/mkdocs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix: Decrase anchor validation treeprocessor prioriy #3945

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix: Decrase anchor validation treeprocessor prioriy #3945

wants to merge 3 commits into from

Conversation

joapuiib
Copy link

Resolves #3690

(...) the suggested solution is to decrease the treeprocessor processor priority to 3.9:

mkdocs/mkdocs/structure/pages.py

Line 343 in fb1d106

md.treeprocessors.register(self, "mkdocs_extract_anchors", priority=5) # Same as 'toc'.

pawamoy
pawamoy reviewed Mar 21, 2025
View reviewed changes
Copy link
Collaborator
@pawamoy pawamoy left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks! More than just tests passing, we should also look out for regressions by building a few projects with this change, asserting there's no issue. I'll do that myself anyway, but you can also do it.

@joapuiib @pawamoy
Update mkdocs/structure/pages.py
c2ec336 
Co-authored-by: Timothée Mazzucotelli <dev@pawamoy.fr>
@joapuiib < 8000 /div>
Copy link
Author

I've tested it with one of my sites that was giving the anchor validation false positive, and it seems that it works with no issues.

However, it might be better if you test it as well, just in case 😅

@byrnereese
Copy link

Nudge. I would love to see this fix get merged. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pawamoy pawamoy pawamoy left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Anchor validation false positives after upgrading to 1.6
3 participants
@joapuiib @byrnereese @pawamoy
0