Skip to content

fix(ci): bump the editorconfig-checker action past the v4 rename - #37

Open
yukimemi wants to merge 1 commit into
mainfrom
fix-ec-action
Open

fix(ci): bump the editorconfig-checker action past the v4 rename#37
yukimemi wants to merge 1 commit into
mainfrom
fix-ec-action

Conversation

@yukimemi

@yukimemi yukimemi commented Sep 3, 2026

Copy link
Copy Markdown
Owner

The editorconfig job has been failing on every consumer repo, and not on a rule violation:

Error: The binary 'ec-linux-amd64*' not found

editorconfig-checker v4.0.0 renamed its release assets from ec-<os>-<arch> to editorconfig-checker-<os>-<arch>. action-editorconfig-checker@v2 still looks for the old name, so it cannot fetch the binary it is meant to run — the job fails before it ever reads an .editorconfig.

v3.0.0 of the action is that fix, released today:

feat: support editorconfig-checker binary name for v4 with legacy ec fallback

How it surfaced

On yukimemi/magi, where the job had been red on every commit since the sweep switched it on — and where it briefly blocked a release, because making editorconfig a required status check turned a broken action into an unmergeable repository.

Pinned to v3.0.0 rather than floated on v3: this pair has now drifted twice, and a tag keeps Renovate in charge of when it moves — which is what the # renovate: comment above the pin is for.

The `editorconfig` job has been failing on every consumer repo with

    Error: The binary 'ec-linux-amd64*' not found

not on a rule violation. `editorconfig-checker` v4.0.0 renamed its release
assets from `ec-<os>-<arch>` to `editorconfig-checker-<os>-<arch>`, and
`action-editorconfig-checker@v2` still looks for the old name, so it cannot
fetch the binary it is supposed to run.

`v3.0.0` of the action is exactly that fix - "support editorconfig-checker
binary name for v4 with legacy `ec` fallback" - and it shipped today.

Found on yukimemi/magi, where the job had gone red on every commit since the
sweep switched it on. Worth pinning rather than floating on `v3`: this is the
second time the pair has drifted, and a tag keeps Renovate in charge of when
it moves.
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: ae0f7b74-7456-4417-b314-401c9aea7bc5


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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