Automatically add regression labels to glacier tracked issues #767
Conversation
There was a problem hiding this comment.
I wouldn't add this functionality.
The problem is that this will cause a lot of notifications, as mentioned in https://rust-lang.zulipchat.com/#narrow/stream/227806-t-compiler.2Fwg-prioritization/topic/adding.20labels.20to.20stable-to-stable.20regressions. And I don't think it makes much sense to label all the ICEs, which produces much work to wg-prioritization. Moreover relabeling is also unnecessary currently IMO. Looking at regression-from-stable-to-nightly and regression-from-stable-to-beta, some members triage them manually and add milestones, it should be enough.
|
But this PR has some unrelated refactoring, e.g. removing regex dependency. I'm happy to accept it. |
|
I didn't realize this would ping prioritization and add so much work for them. I agree that it shouldn't notify them on every ice added to glacier. Should #568 be closed then? |
Fixes #568