Repository navigation
Vale from remote repo - #35
Merged
Merged
Conversation
m4sterbunny
changed the title
lucas-tortora
approved these changes
Oct 5, 2026
Contributor
Author
|
Keeping branch persistent |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Troubleshooting fixes for vale runs from remote repo relying on this branch:
Fixed
shell: bashruns with -e, so an exit of 1 from Vale ended the stepbefore the annotations were printed or the advisory decision ran, leaving
a red job with an empty log. Disable errexit for the Vale call and run the
summary whenever files were linted.
Vale already skips link targets and lints link text as prose. The regex
version desynchronised the parser, so a link whose text is inline code
was linted in both the text and the URL.
Add maintainer support for remote config setup
Drop the inline-code and fenced-block ignore patterns: Vale's Markdown parser already skips inline code and fenced blocks. The regex versions mis-pair backticks across a file, so Vale linted inside fences and reported prose on wrong lines.
Co-Authored-By: Claude Fable 5.1