Repository navigation
feat(evals): link Braintrust experiments to their CI run and the PR - #384
Merged
mattrossman merged 31 commits intoOct 9, 2026
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
This reverts commit 43f5cfe.
This reverts commit 9c75553.
This reverts commit 7a71bd8.
This reverts commit 85b9c81.
mattrossman
marked this pull request as ready for review
October 9, 2026 17:44
gregnr
approved these changes
Oct 9, 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.
Improves cross-linking between Braintrust <> GitHub Actions <> GitHub PR. This makes it easier to e.g. debug CI logs from a trace, or debug the source of a run.
ci_run_urlwhen relevant GitHub env vars are presentNote
Manual workflows are associated with branches, not PRs directly. The job summary lists the PR that triggered a PR-synced run (like with
run-evals-changed), and for a manual dispatch it lists every open PR for the target branch, or(none).ci_run_urlin ExperimentThe existing
pr_urlin Braintrust metadata is sourced only for PR triggers, so it doesn't map cleanly to the way I check for multiple PRs in the job summary. I've left it as-is to avoid overloading concepts.Closes AI-1301