Skip to content

Make comment of triage build link back to the triage invocation - #2543

Merged
JonathanBrouwer merged 3 commits into
rust-lang:mainfrom
JonathanBrouwer:triage-run-detection
Aug 22, 2026
Merged

Make comment of triage build link back to the triage invocation#2543
JonathanBrouwer merged 3 commits into
rust-lang:mainfrom
JonathanBrouwer:triage-run-detection

Conversation

@JonathanBrouwer

@JonathanBrouwer JonathanBrouwer commented Aug 21, 2026

Copy link
Copy Markdown
Member

r? @Kobzol

Recommend reviewing commit by commit.
Again the "did not test this" disclaimer other than what I mentioned in the self-review

Comment thread site/src/github/client.rs Outdated
Comment thread site/src/github/comparison_summary.rs

@Kobzol Kobzol left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable. There's not really an easy way to test this outside of production.

Comment thread site/src/github/triage.rs Outdated

@Kobzol Kobzol left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please get rid of the merge commit. Also suggested a rewording. Otherwise LGTM.

Comment thread site/src/github/comparison_summary.rs Outdated
@JonathanBrouwer

Copy link
Copy Markdown
Member Author

Oops sorry for that merge commit 💀

@JonathanBrouwer
JonathanBrouwer added this pull request to the merge queue Aug 22, 2026
Merged via the queue into rust-lang:main with commit a7e3e84 Aug 22, 2026
15 checks passed
@JonathanBrouwer
JonathanBrouwer deleted the triage-run-detection branch August 22, 2026 12:12
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