Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ICE reporting link template not working. #107145

Closed
recmo opened this issue Jan 21, 2023 · 5 comments
Closed

ICE reporting link template not working. #107145

recmo opened this issue Jan 21, 2023 · 5 comments
Labels
A-diagnostics Area: Messages for errors, warnings, and lints T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@recmo
Copy link

recmo commented Jan 21, 2023

Code

(any code triggering ICE)

Current output

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

Desired output

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?assignees=&labels=C-bug%2CI-ICE%2CT-compiler&template=ice.yaml&title=%5BICE%5D%3A+

Rationale and extra context

The provided link opens an empty issue and does not set the template (as the url would suggest).

Other cases

No response

Anything else?

No response

@recmo recmo added A-diagnostics Area: Messages for errors, warnings, and lints T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jan 21, 2023
@albertlarsan68
Copy link
Member

Cc #106831

@ehuss
Copy link
Contributor

ehuss commented Jan 21, 2023

cc @estebank

@Noratrieb
Copy link
Member

#106898 brings it back

@estebank
Copy link
Contributor

Apologies, I've had to leave suddenly and am going to be unavailable for a another couple of weeks. I would appreciate it if someone could follow up on this. I've moved to merge my PR, but I'd feel more comfortable if someone kept an eye out for further problems here. Apologies for introducing these growing pains.

@jyn514
Copy link
Member

jyn514 commented Feb 13, 2023

#106898 was merged. I think it might make sense to open an issue for following up here, but I don't think we need to keep this issue open.

@jyn514 jyn514 closed this as completed Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-diagnostics Area: Messages for errors, warnings, and lints T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

6 participants