Skip to content

fix: vale-ignore for reST#184

Open
medubelko wants to merge 2 commits intocanonical:mainfrom
medubelko:docpr-2560-fix-vale-ignore
Open

fix: vale-ignore for reST#184
medubelko wants to merge 2 commits intocanonical:mainfrom
medubelko:docpr-2560-fix-vale-ignore

Conversation

@medubelko
Copy link
Contributor

Resolves #183.

Add missing capture group.

Also, reorganise the sections in the config more logically.
- id: ignore-rst
filetypes: [rst]
content: |
Ignore the :vale-ignore:`splelling` mistake.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should I add all the other roles? I'm ambivalent as to whether the vale-ignore is more about spelling or all of Vale.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we need the other roles. I think if there are issues we can implement a fix for them - but I haven't seen any reports of the other roles causing issues with linting (though we'll actually see that later when these rules are properly enabled).

Hmm, I actually think we should remove woke-ignore - as it's functionally exactly the same and isn't actually restricted to just the inclusive language rule.

@medubelko medubelko force-pushed the docpr-2560-fix-vale-ignore branch from 610be1a to f1a9287 Compare March 6, 2026 03:50
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.

vale-ignore doesn't work for reST

2 participants