Skip to content

feat(ci): autofix in CI instead of requiring contributors to push a new commit#6574

Merged
CommanderStorm merged 3 commits intolouislam:masterfrom
CommanderStorm:add-a-formatting-fix-action
Jan 3, 2026
Merged

feat(ci): autofix in CI instead of requiring contributors to push a new commit#6574
CommanderStorm merged 3 commits intolouislam:masterfrom
CommanderStorm:add-a-formatting-fix-action

Conversation

@CommanderStorm
Copy link
Copy Markdown
Collaborator

ℹ️ To keep reviews fast and effective, please make sure you’ve read our pull request guidelines

📝 Summary of changes done and why they are done

Instead of requiring me or contributors to fix pure formatting issues, we should just do this for them.

@louislam this again requires admin level priviliges to enable.
-> please install this here https://autofix.ci/

to see it in action in CommanderStorm/mdbook-mermaid-ssr#44

  • A contributor creates a change with a formatting issue (in this case updating the version in one, but not the other place)
  • The action fixes this
  • A commit is pushed

All of this is much simpler to get secure than the alternatives (where I would not know if i can entirely make that happen) that I could build.
In terms of actual alternatives, I only know of http://pre-commit.ci/ and I like this more, since it is more out of the way.

Copilot AI review requested due to automatic review settings January 3, 2026 00:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces an automated code formatting workflow using autofix.ci to automatically fix linting issues in pull requests, eliminating the need for contributors to manually push formatting fixes.

Key changes:

  • Adds a new GitHub Actions workflow that runs ESLint and Stylelint auto-fixers on pushes and pull requests
  • Integrates with autofix.ci to automatically commit formatting fixes back to the branch

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@CommanderStorm CommanderStorm added the pr:depends on other pending other things to be done first label Jan 3, 2026
@louislam
Copy link
Copy Markdown
Owner

louislam commented Jan 3, 2026

-> please install this here https://autofix.ci/

Should be just installed.

@CommanderStorm CommanderStorm changed the title feat: autofix in CI instead of requiring contributors to push a new commit feat(ci): autofix in CI instead of requiring contributors to push a new commit Jan 3, 2026
@CommanderStorm CommanderStorm added this to the 2.1.0 milestone Jan 3, 2026
@CommanderStorm CommanderStorm merged commit 531748d into louislam:master Jan 3, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:depends on other pending other things to be done first

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants