Skip to content

ci: add GitHub Actions workflow for PR validation - #252

Merged
yanglbme merged 1 commit into
mainfrom
ci/github-actions
Jun 17, 2026
Merged

yanglbme merged 1 commit into
mainfrom
ci/github-actions

Conversation

@yanglbme

Copy link
Copy Markdown
Member

Summary

  • Add a CI GitHub Actions workflow for pull_request and push to main
  • Run pnpm install --frozen-lockfile, Chromium build, artifact checks, unit tests, Firefox build, and web-ext lint
  • Use read-only permissions and no repository secrets so fork PRs can run the same checks

Fork PR notes

  • Uses the standard pull_request event (not pull_request_target)
  • No secrets or write permissions required
  • First-time contributors from forks may still need maintainer approval before GitHub runs workflows (repository setting)

Test plan

  • Local: pnpm install --frozen-lockfile
  • Local: pnpm build + artifact checks
  • Local: pnpm test
  • Local: pnpm build:firefox + pnpm -C apps/extension lint
  • CI passes on this PR after merge workflow is available on main

Run install, Chromium build, tests, and Firefox lint on pull_request
and main pushes without secrets so fork PRs can use the same checks.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings June 17, 2026 11:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@yanglbme
yanglbme merged commit 5a7d170 into main Jun 17, 2026
1 check passed
@yanglbme
yanglbme deleted the ci/github-actions branch June 17, 2026 11: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.

2 participants