Skip to content

Format and update actions#193

Merged
davidbuzinski merged 2 commits intomainfrom
format-and-update-actions
Apr 7, 2026
Merged

Format and update actions#193
davidbuzinski merged 2 commits intomainfrom
format-and-update-actions

Conversation

@davidbuzinski
Copy link
Copy Markdown
Contributor

@davidbuzinski davidbuzinski commented Apr 6, 2026

This change updates our CI workflows to use v3 of our actions, and adds a format gate in CI. I added steps for how to setup a git pre-commit hook to run checks to the devel/contributing.md file.

Comment thread package.json
"build": "tsc",
"package": "ncc build src/index.ts -o dist/setup --minify && ncc build src/post.ts -o dist/cache-save --minify",
"test": "NODE_OPTIONS='--experimental-vm-modules' jest",
"all": "npm run lint && npm test && npm run build && npm run package",
Copy link
Copy Markdown
Contributor Author

@davidbuzinski davidbuzinski Apr 7, 2026

Choose a reason for hiding this comment

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

This is the only substantial change I made to the action in this change (other than the actions versions in the workflow). This will fail a CI job if there's formatter findings.

Copy link
Copy Markdown
Contributor

@sameagen-MW sameagen-MW left a comment

Choose a reason for hiding this comment

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

LGTM, thanks David!!

:shipit:

@davidbuzinski davidbuzinski merged commit 245ed85 into main Apr 7, 2026
11 checks passed
@davidbuzinski davidbuzinski deleted the format-and-update-actions branch April 7, 2026 13:58
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