Skip to content

chore(ci): make the pull request title check faster - #3779

Merged
felipefreitag merged 1 commit into
canaryfrom
chore/faster-pr-title-check
Oct 8, 2026
Merged

felipefreitag merged 1 commit into
canaryfrom
chore/faster-pr-title-check

Conversation

@felipefreitag

@felipefreitag felipefreitag commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

The pull request title check now finishes in seconds, and a quick body edit no longer delays it by about 5 minutes. Before, an edit soon after a push cancelled the running check. That cancel took about 5 minutes on Depot, and the new run waited for it.

Not covered. Other workflows still cancel on a new push and can hit the same slow cancel. This one hit it far more often, because only it runs on body edits.

Decision. The script runs with plain Node 24 instead of a package install. Node 24 strips the types itself, so the check needs no dependencies.


Summary by cubic

Makes the pull request title check finish in seconds so a quick body edit no longer delays it by about five minutes. Before, an edit soon after a push cancelled the running check, the cancel took about five minutes on Depot, and the new run waited for it.

The check now runs with plain Node 24 instead of a pnpm install and tsx; Node 24 strips types itself, so the check needs no dependencies. Other workflows still cancel on a new push and can hit the same slow cancel, but only this one ran on body edits.

Written for commit ee1d17c. Summary will update on new commits.

View guided diff

Remove the concurrency group. A body edit soon after a push cancelled the
running check, the cancel took about 5 minutes, and the new run waited for it.

Run the script with plain Node instead of a full pnpm install and tsx.
@vercel

vercel Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
react-email Ready Ready Preview Oct 8, 2026 1:26pm UTC
react-email-demo Ready Ready Preview Oct 8, 2026 1:26pm UTC

Request Review

@changeset-bot

changeset-bot Bot commented Oct 8, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: ee1d17c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 1 file

Reply with feedback, questions, or to request a fix.

View guided diff | Re-trigger cubic

Comment thread .github/workflows/pull-request-title-check.yml
@felipefreitag
felipefreitag merged commit 1531a39 into canary Oct 8, 2026
15 checks passed
@felipefreitag
felipefreitag deleted the chore/faster-pr-title-check branch October 8, 2026 13:45

This branch was successfully deployed

2 active deployments
Preview – react-email-demo — ee1d17cf Deployed Oct 8, 2026 by vercel[bot]
Preview – react-email — ee1d17cf Deployed Oct 8, 2026 by vercel[bot]
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.

2 participants