Skip to content

Enable triagebot issue-links - #7098

Merged
ytmimi merged 1 commit into
rust-lang:mainfrom
jieyouxu:canon-issue-links
Sep 4, 2026
Merged

ytmimi merged 1 commit into
rust-lang:mainfrom
jieyouxu:canon-issue-links

Conversation

@jieyouxu

@jieyouxu jieyouxu commented Sep 3, 2026

Copy link
Copy Markdown
Member

Noticed in #7095 (comment), triagebot has a functionality to:

  • Automatically canonicalize issue links in PR description, and
  • Show a warning message if there are Fixes #123456 in commit descriptions.

The Fixes #123456 form triggers github magic comments, which will close the same issue number / PR but on the wrong repo when syncing subtrees.


  • I did not use an LLM to create a change in this PR.
  • I used an LLM to create a change in this PR, and I have explained below how it was used.

@jieyouxu jieyouxu added the A-meta Area: meta (e.g. triagebot configuration) label Sep 3, 2026
@rustbot rustbot added the S-waiting-on-review Status: awaiting review from the assignee but also interested parties. label Sep 3, 2026
@jieyouxu jieyouxu mentioned this pull request Sep 3, 2026
2 tasks

@ytmimi ytmimi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

First, this is a really cool feature and I'm glad that we're enabling it 🎉. I think issue links in commits are fine, they should just be conical to rust-lang/rustfmt.

View changes since this review

Comment thread triagebot.toml
# Canonicalize issue numbers to avoid closing the wrong issue
# when commits are included in subtrees, as well as warning links in commits.
# Documentation at: https://forge.rust-lang.org/triagebot/issue-links.html
[issue-links]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Per the docs I think we should also add check-commits = "uncanonicalized" # for subtrees

Suggested change
[issue-links]
[issue-links]
# This prevents links from resolving to the wrong repository when the subtree is merged upstream while still allowing issue links in commits.
check-commits = "uncanonicalized"

@rustbot rustbot added S-waiting-on-author Status: awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: awaiting review from the assignee but also interested parties. labels Sep 3, 2026
@jieyouxu

jieyouxu commented Sep 4, 2026

Copy link
Copy Markdown
Member Author

Changes since last review:

  • Added check-commits = "uncanonicalized"

@rustbot review

@rustbot rustbot added S-waiting-on-review Status: awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: awaiting some action (such as code changes or more information) from the author. labels Sep 4, 2026
@ytmimi
ytmimi added this pull request to the merge queue Sep 4, 2026
Merged via the queue into rust-lang:main with commit 2d897e2 Sep 4, 2026
27 checks passed
@rustbot rustbot added release-notes Needs an associated changelog entry and removed S-waiting-on-review Status: awaiting review from the assignee but also interested parties. labels Sep 4, 2026
@jieyouxu
jieyouxu deleted the canon-issue-links branch September 9, 2026 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-meta Area: meta (e.g. triagebot configuration) release-notes Needs an associated changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants