Skip to content

ci: run editorconfig-checker - #205

Merged
dangreen merged 2 commits into
mainfrom
ci/run-editorconfig-checker
Oct 19, 2026
Merged

dangreen merged 2 commits into
mainfrom
ci/run-editorconfig-checker

Conversation

@dangreen

@dangreen dangreen commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

Why

The editorconfig job installs editorconfig-checker but never runs it: the main branch of editorconfig-checker/action-editorconfig-checker only adds the binary to PATH, and the example workflow of the action runs it in a step of its own. The job moved from @v1, a Docker action that ran the check by itself, to @main in eceaf52 without such a step, so it has not checked a file since June 2023.

Run as is, editorconfig-checker v4.0.2 reports 3 errors, all from its indent-size rule, which wants every left-padding to be a multiple of indent_size. They are lines of numbered list items in skills/codex/SKILL.md and skills/project/SKILL.md, which markdown aligns with the text of the item, 3 spaces after 1. , and a wrapped row of the file table in skills/project/SKILL.md.

What

  • Numbered list items with nested content are written as 1. followed by two spaces, so the content sits at 4 spaces instead of 3. The second column of the file table in skills/project/SKILL.md moves one space right, to an even column.
  • The step with the action is now named Install editorconfig-checker, and a new Check editorconfig step runs editorconfig-checker.

Notes

  • .editorconfig stays as it is, so every file is still checked against indent_size.
  • skills/codex/SKILL.md parses to the same tree as before, and skills/project/SKILL.md differs in the file table alone, checked with mdast-util-from-markdown and its GFM extension.
  • With these changes, editorconfig-checker v4.0.2 finds no errors on main.

@dangreen
dangreen force-pushed the ci/run-editorconfig-checker branch from 3416ecd to a7d0c9b Compare October 18, 2026 13:01
@dangreen dangreen changed the title ci: run editorconfig-checker without the indent-size rule ci: run editorconfig-checker Oct 18, 2026
@dangreen
dangreen force-pushed the ci/run-editorconfig-checker branch from a7d0c9b to 3791566 Compare October 18, 2026 13:40
@dangreen
dangreen merged commit 525e5c2 into main Oct 19, 2026
6 checks passed
@dangreen
dangreen deleted the ci/run-editorconfig-checker branch October 19, 2026 10:28
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