Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added CLI option to automatically fix errors 🚁 #16

Merged
merged 9 commits into from
Jul 24, 2024
Merged

Added CLI option to automatically fix errors 🚁 #16

merged 9 commits into from
Jul 24, 2024

Conversation

01taylop
Copy link
Owner

@01taylop 01taylop commented Jul 23, 2024

Details

What have you changed?

  • Added CLI option to automatically fix errors. This includes all three linters; eslint, stylelint, and markdownlint.

Why are you making these changes?

  • So that users can have all the fixable errors fixed for them!
  • The markdownlint fix option behaves differently than the other two in that both eslint and stylelint return results after the fixes have been made. To make the behaviour consistent in Lint Pilot, markdownlint has to be run a second time.

01taylop added 7 commits July 23, 2024 11:49
Unlike the other linters, markdownlint does not fix errors whilst
linting which means the fixed files should be re-linted and the results
updated.
This is so that the results displayed to the user will reflect the
fixes that have been applied.
@01taylop 01taylop changed the title Fix Added CLI option to automatically fix errors 🚁 Jul 24, 2024
@01taylop 01taylop marked this pull request as ready for review July 24, 2024 17:07
@01taylop 01taylop merged commit 71bafe2 into main Jul 24, 2024
5 checks passed
@01taylop 01taylop deleted the fix branch July 24, 2024 17:37
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.

1 participant