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

Enable triagebot issue-links#7098
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
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