Skip to content

Fix CI test to check for actual script output#6

Merged
alexeagle merged 1 commit intoaspect-extensions:mainfrom
bamcmanus:fix/ci-test-expected-output
Jan 30, 2026
Merged

Fix CI test to check for actual script output#6
alexeagle merged 1 commit intoaspect-extensions:mainfrom
bamcmanus:fix/ci-test-expected-output

Conversation

@bamcmanus
Copy link
Contributor

Problem

The CI test has never passed because it checks for "SUCCESS" in the output, but
bazel-diff.axl never outputs that string.

Solution

Change the test to grep for "Impacted Targets" instead. This is the final
message teh script outputs on successful completion.

Testing

Ran ./README.md locally and verified the test passes.

@alexeagle alexeagle merged commit 2aa548d into aspect-extensions:main Jan 30, 2026
0 of 2 checks passed
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