Skip to content

Docs: Updates custom linter rules documentation#1773

Merged
marcoroth merged 1 commit into
marcoroth:mainfrom
jkogara:jkogara/update_documentation
Jun 4, 2026
Merged

Docs: Updates custom linter rules documentation#1773
marcoroth merged 1 commit into
marcoroth:mainfrom
jkogara:jkogara/update_documentation

Conversation

@jkogara

@jkogara jkogara commented Jun 3, 2026

Copy link
Copy Markdown
Contributor
  • Custom linter rule examples use this.name when constructing visitors, but ParserRule exposes the rule name via this.ruleName.
  • Using this.name results in undefined being passed to the visitor, causing the rule name to show as undefined in CLI output and breaking (at least my nvim) LSP diagnostics.

@github-actions github-actions Bot added the linter label Jun 3, 2026

@marcoroth marcoroth left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks so much for catching this @jkogara! 🙏🏼

@marcoroth marcoroth merged commit 6a336cc into marcoroth:main Jun 4, 2026
6 of 7 checks passed
@marcoroth marcoroth changed the title Updates custom linter rules documentation Docs: Updates custom linter rules documentation Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants