Skip to content

ci: add CI + lint-pr-title workflows#51

Merged
mjdavidson merged 1 commit into
masterfrom
mattd/explore-add-ci
May 7, 2026
Merged

ci: add CI + lint-pr-title workflows#51
mjdavidson merged 1 commit into
masterfrom
mattd/explore-add-ci

Conversation

@mjdavidson
Copy link
Copy Markdown
Contributor

Adds ci.yml (install, lint, format:check, build on push/PR) and lint-pr-title.yml (enforces conventional-commit PR titles via amannn/action-semantic-pull-request).

ci.yml runs on push to master and PRs: install, lint, format:check,
build. Mirrors what AGENTS.md tells contributors to run before
committing.

lint-pr-title.yml enforces conventional-commit-style PR titles
(feat:, fix:, chore:, etc.) so the merge log stays consistent.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 7, 2026

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

Project Deployment Actions Updated (UTC)
explore Ready Ready Preview, Comment May 7, 2026 5:23pm

Request Review

@mjdavidson mjdavidson requested a review from mattgd May 7, 2026 17:25
@mjdavidson mjdavidson merged commit 8a295a0 into master May 7, 2026
8 checks passed
@mjdavidson mjdavidson deleted the mattd/explore-add-ci branch May 7, 2026 20:26
lint:
runs-on: ubuntu-latest
steps:
- uses: amannn/action-semantic-pull-request@v6
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do we use this in other repos?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes — amannn/action-semantic-pull-request is in lint-pr-title.yml across workos-node, authkit-nextjs, authkit-react, workos-ruby, workos-go, workos-python, and workos-php (all 8 public SDK/integration repos I sampled). Example: authkit-nextjs/lint-pr-title.yml.

One nit I should fix to match the pattern more closely: those repos pin the action to a SHA (@48f256284bd46cdaab1048c3721360e808335d50 # v6) rather than the floating @v6 tag. Happy to tighten the pin in this PR or as a follow-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants