Skip to content

ci: skip scheduled and release workflows on forks - #5467

Open
batmnnn wants to merge 1 commit into
boa-dev:mainfrom
batmnnn:ci/skip-workflows-on-forks
Open

ci: skip scheduled and release workflows on forks#5467
batmnnn wants to merge 1 commit into
boa-dev:mainfrom
batmnnn:ci/skip-workflows-on-forks

Conversation

@batmnnn

@batmnnn batmnnn commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add if: github.repository == 'boa-dev/boa' to Security audit, Nightly Build, Update Test262 Results, CodeQL, and Publish Release jobs
  • Prevents forks with Actions enabled from running workflows that require upstream secrets (DATA_PAT, registry tokens) or write permissions, which currently fail and spam contributors with notifications

Test plan

  • Confirm jobs still run on boa-dev/boa (condition evaluates true)
  • On a fork with Actions enabled, these workflows should show as skipped rather than failed

Made with Cursor

These jobs need upstream secrets or write access and fail noisily when Actions are enabled on a fork.

Co-authored-by: Cursor <cursoragent@cursor.com>
@batmnnn
batmnnn requested a review from a team as a code owner July 30, 2026 21:10
Copilot AI review requested due to automatic review settings July 30, 2026 21:10

Copilot AI 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.

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

@github-actions github-actions Bot added Waiting On Review Waiting on reviews from the maintainers C-Tests Issues and PRs related to the tests. C-Actions Pull requests that update Github Actions code labels Jul 30, 2026
@github-actions github-actions Bot added this to the v1.0.0 milestone Jul 30, 2026
@github-actions

Copy link
Copy Markdown

Test262 conformance changes

Test result main count PR count difference
Total 53,125 53,125 0
Passed 51,073 51,073 0
Ignored 1,482 1,482 0
Failed 570 570 0
Panics 0 0 0
Conformance 96.14% 96.14% 0.00%

Tested main commit: 4fc75c6ae9d85f2b8065c6716f88e9b35318438c
Tested PR commit: 22fd5dc86780b4f055e642108f515f84aca3b8c1
Compare commits: 4fc75c6...22fd5dc

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.88%. Comparing base (6ddc2b4) to head (22fd5dc).
⚠️ Report is 1012 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #5467       +/-   ##
===========================================
+ Coverage   47.24%   62.88%   +15.63%     
===========================================
  Files         476      530       +54     
  Lines       46892    59102    +12210     
===========================================
+ Hits        22154    37165    +15011     
+ Misses      24738    21937     -2801     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

C-Actions Pull requests that update Github Actions code C-Tests Issues and PRs related to the tests. Waiting On Review Waiting on reviews from the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants