Skip to content

Allow anchors to be ignored for specific URLs#11489

Merged
AA-Turner merged 7 commits intosphinx-doc:masterfrom
picnixz:feature/11484-linkcheck-blacklist
Jul 24, 2023
Merged

Allow anchors to be ignored for specific URLs#11489
AA-Turner merged 7 commits intosphinx-doc:masterfrom
picnixz:feature/11484-linkcheck-blacklist

Conversation

@picnixz
Copy link
Copy Markdown
Member

@picnixz picnixz commented Jul 16, 2023

Closes #11484

Copy link
Copy Markdown
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Very clean, thank you!

@picnixz picnixz force-pushed the feature/11484-linkcheck-blacklist branch 2 times, most recently from 79d9af7 to 3ca85aa Compare July 18, 2023 07:00
@picnixz picnixz force-pushed the feature/11484-linkcheck-blacklist branch from 3ca85aa to 3e57128 Compare July 18, 2023 07:01
@AA-Turner
Copy link
Copy Markdown
Member

@picnixz -- this has conflicts now, please may you resolve?

A

@picnixz picnixz mentioned this pull request Jul 24, 2023
@jayaddison
Copy link
Copy Markdown
Contributor

Checking my understanding of this feature, as a non-blocking comment: this feature essentially trims the anchor component of matching URL patterns, resulting in the containing page URL being checked, with presence of the anchor unchecked.

@picnixz
Copy link
Copy Markdown
Member Author

picnixz commented Jul 24, 2023

Thank you for reorganizing the CHANGELOG (from one PR to another, I always forget whether I should put it at the top or the bottom of the changelist 😅) and correcting my English !

@AA-Turner
Copy link
Copy Markdown
Member

No worries, it's a subtle feature to describe -- I went with "skip verifying that the anchors exist" as I think it captures what/why a user would gain from enabling the feature.

I'm going to do a review of CHANGES just now before the release (hopefully later today), so will fix order then -- I tend to go for the order in which features were added.

A

@AA-Turner AA-Turner merged commit 82bc15a into sphinx-doc:master Jul 24, 2023
@picnixz picnixz deleted the feature/11484-linkcheck-blacklist branch July 24, 2023 11:16
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow disabling linkcheck anchor checks for specific URLs

4 participants