Skip to content

fix: keep fork quality workflows running - #524

Open
Onur (onuracar-dev) wants to merge 1 commit into
microsoft:mainfrom
onuracar-dev:fix/522-fork-safe-quality-comments
Open

fix: keep fork quality workflows running#524
Onur (onuracar-dev) wants to merge 1 commit into
microsoft:mainfrom
onuracar-dev:fix/522-fork-safe-quality-comments

Conversation

@onuracar-dev

Copy link
Copy Markdown

Summary

  • skip sticky PR-comment steps for fork pull requests while still running performance and coverage checks
  • keep comment publishing non-gating for same-repository pull requests
  • include generated Markdown reports alongside raw snapshot artifacts
  • add regression coverage for every quality-workflow comment step

The snapshot comparison steps remain fail-closed; only comment publishing uses continue-on-error.

Fixes #522

Testing

  • python -B -m unittest discover -s scripts/tests -p 'test_*.py' -v (49 tests)
  • cargo fmt --all -- --check
  • actionlint .github/workflows/coverage.yml .github/workflows/perf-tests.yml

@onuracar-dev

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

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.

Make quality snapshot comments safe for fork PRs

1 participant