Skip to content

Add informational Nx affected PR checks#7812

Closed
isaacroldan wants to merge 1 commit into
mainfrom
ir/cli-perf-affected-ci
Closed

Add informational Nx affected PR checks#7812
isaacroldan wants to merge 1 commit into
mainfrom
ir/cli-perf-affected-ci

Conversation

@isaacroldan

@isaacroldan isaacroldan commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

WHY are these changes introduced?

We want a safe comparison path for Nx affected CI timing and coverage before replacing the existing full PR checks.

WHAT is this pull request doing?

Adds a PR-only Affected checks (informational) job that:

  • checks out the PR head SHA
  • fetches the PR base SHA explicitly so nx affected can compute the changed graph from a shallow checkout
  • runs affected build, lint, and type-check with continue-on-error
  • writes the target outcomes to the GitHub step summary

The existing full required PR jobs remain unchanged and authoritative. The affected job is intentionally non-blocking and does not run for merge_group.

How to test your changes?

  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/tests-pr.yml"); puts "YAML OK"'
  • git diff --check

Full CI was not run locally.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jun 15, 2026
@isaacroldan

Copy link
Copy Markdown
Contributor Author

Closing this one: we do not need the extra informational affected-check job right now.

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

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant