Skip to content

Linter: Implement Autofix for html-no-space-in-tag linter rule#642

Merged
marcoroth merged 1 commit intomainfrom
autofix-html-no-space-in-tag
Oct 14, 2025
Merged

Linter: Implement Autofix for html-no-space-in-tag linter rule#642
marcoroth merged 1 commit intomainfrom
autofix-html-no-space-in-tag

Conversation

@marcoroth
Copy link
Owner

@marcoroth marcoroth commented Oct 14, 2025

This pull request implements the autofix function for the html-no-space-in-tag linter rule, so that this rule can be autocorrected when running the Herb Linter CLI using --fix.

@marcoroth marcoroth added feature New feature or request linter linter-rule labels Oct 14, 2025
@marcoroth marcoroth merged commit 209b45e into main Oct 14, 2025
7 checks passed
@marcoroth marcoroth deleted the autofix-html-no-space-in-tag branch October 14, 2025 20:34
marcoroth added a commit that referenced this pull request Oct 20, 2025
The `html-no-space-in-tag` linter rule (#559 and #642) has quite a few
false positives and corrupts documents when using the `--fix` option
(see #695), which is why this pull request removes the
`html-no-space-in-tag` rule from the default rules for now.

We can enable this rule again in the future when we improve the accuracy
of the rule.
asilano pushed a commit to fac/herb that referenced this pull request Oct 21, 2025
…coroth#642)

This pull request implements the autofix function for the
`html-no-space-in-tag` linter rule, so that this rule can be
autocorrected when running the Herb Linter CLI using `--fix`.
asilano pushed a commit to fac/herb that referenced this pull request Oct 21, 2025
…coroth#697)

The `html-no-space-in-tag` linter rule (marcoroth#559 and marcoroth#642) has quite a few
false positives and corrupts documents when using the `--fix` option
(see marcoroth#695), which is why this pull request removes the
`html-no-space-in-tag` rule from the default rules for now.

We can enable this rule again in the future when we improve the accuracy
of the rule.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request linter linter-rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant