Skip to content

Fix verify-action-build to try 'npm run start' for actions that use it#664

Merged
raboof merged 1 commit intomainfrom
fix-verify-build-start-script
Apr 5, 2026
Merged

Fix verify-action-build to try 'npm run start' for actions that use it#664
raboof merged 1 commit intomainfrom
fix-verify-build-start-script

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 4, 2026

Some actions like editorconfig-checker/action-editorconfig-checker use "start" as their build script name. The build detection previously only tried "build" and "package" before falling back to a bare npx ncc build which fails for TypeScript entry points.

Generated-by: Claude Opus 4.6 (1M context) noreply@anthropic.com

Some actions like editorconfig-checker/action-editorconfig-checker use
"start" as their build script name. The build detection previously only
tried "build" and "package" before falling back to a bare `npx ncc build`
which fails for TypeScript entry points.

Generated-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 4, 2026

I literally asked "please check why editorconfig-checker/action-editorconfig-checker does not work, Claude figured it out on its own.

@dave2wave dave2wave removed their assignment Apr 4, 2026
@dave2wave
Copy link
Copy Markdown
Member

@potiuk I don't understand why you assign PRs to me that you are writing. If you want me to review then use that.

Copy link
Copy Markdown
Member

@dave2wave dave2wave left a comment

Choose a reason for hiding this comment

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

Looks good to me

@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 4, 2026

@potiuk I don't understand why you assign PRs to me that you are writing. If you want me to review then use that.

It was just setting reviewers, not assigning. I thought engaged people might be interested in doing reviews here :). That's what we usually do in other projects - letting other people.who we think are interested in reviewing new things.

This can also be automated with CODEOWNERS, i am not sure what is the best way here .

@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 4, 2026

Oh that was an honest mistake. To be honest GitHub interface makes it very easy to choose wrong option (happened in a past).

Sorry for that

@raboof raboof merged commit 6eafdbc into main Apr 5, 2026
7 checks passed
@raboof raboof deleted the fix-verify-build-start-script branch April 5, 2026 10:39
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.

3 participants