Add ACT4 nightly compliance test workflow#631
Merged
Conversation
…, gate on upstream success, validate SHA
Collaborator
Author
|
/claude /codex |
Collaborator
Author
|
/claude /codex |
Codex Code ReviewNo findings. The PR only adds |
The test job was gated on the "Bench Vs Nightly" run concluding with success. Since the workflow_run trigger is only used to serialize on the shared bench runner (not a real dependency), a failed or timed-out benchmark silently skipped ACT4 for the night and filed no issue. Run on any benchmark conclusion, skipping only 'cancelled' (a newer bench run is already inbound) and 'skipped' (nothing ran).
A GitHub issue is easy to miss. Post nightly ACT4 failures to the same Slack channel the nightly benchmark already reports to, reusing BENCH_VS_SLACK_WEBHOOK (and the _TEST variant for manual runs so smoke tests don't ping the real channel). The report job no longer needs issues: write. The payload is built with jq so the commit SHA and result summary are safely encoded, and the step skips gracefully when the webhook secret is absent.
MauroToscano
approved these changes
Jun 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.