Repository navigation
Show Run results in notification email bodies - #575
Merged
Merged
Conversation
Merged
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.
Summary
Put the Run result first in every Run notification body so it stays visible when a long issue title truncates the subject.
+Result: merged Pull request: https://github.com/<owner>/<repo>/pull/<n> Session log: <path> Command log: <path>Closes #572
Evidence
cargo test --test run_notification a_merge_run_sends_one_notification_that_it_merged -- --exactfailed because the delivered email body started withPull request:and contained no result.After: the same long-title regression passes with
Result: mergedat the start of the delivered body.cargo test: 1,516 passed, 0 failed, 1 ignored.cargo check --all-targets,cargo fmt --check, andcargo clippy --all-targets -- -D warningspassed.Test seams
Unaddressed findings
Both axes reviewed
main...411cfc6(main:ace41a0). Changed files left unread by review: none on either axis; both read all six changed files.Merge Danger
Door: two-way
Reverting removes the added body line; there are no data or configuration changes.
Blast Radius: notifications
Run, Spec run, Architect run and Pickup run email bodies gain a leading result line.
Built with codex · gpt-6.1-sol · xhigh