-
Notifications
You must be signed in to change notification settings - Fork 300
Archive inactive compiler WGs #1763
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
Conversation
5a7feba
to
d5281d6
Compare
cc @Kobzol / @marcoieni I'm not sure how to fix this CI error 🤔
(I have just moved the team |
There is a wg-debugging repo, that should also be archived (and its Btw if the "working group" suffix is dropped, it might look a bit weird on the website (it will show just "Binary size" as the header). |
I tried to improve the error message in #1764 👍 |
I get a new error locally, unrelated to this PR:
Should I fix this first in a separate PR? What do we do in these cases? |
Yeah, we should do that in a separate PR. It's a bit unfortunate, but when a user in the team DB renames themselves, it currently blocks |
Must be my lucky day. Exactly while I was changing stuff here :D |
d5281d6
to
4723d6d
Compare
4723d6d
to
80ae8fe
Compare
Dry-run check results
|
In case this is ready, probably a team-repo-admin should review this (not infra admins) since it changes teams 👍 |
80ae8fe
to
feba264
Compare
I thought it was ok but since you mention maybe it was not that ok, so I have added a "working area" suffix for some of them, which is not great either but it feels better (and I am uninspired to find a better title anyway). At least it's coherent with what we have in the forge |
Yeah, it's hard to find a good name. Working group actually was a pretty good match 😆 Working area sounds fine. |
(particulary on "working area" naming, I guess) |
I'd probably make the remaining groups into marker teams so that they're just ping lists - I think that is what we're intending with these changes. |
dad073c
to
874e8ff
Compare
@Kobzol I think we're done here (to be clear: I've intentionally only touched files marked as compiler subteams) |
874e8ff
to
dbef395
Compare
Fine by me :) (I can't approve it though) |
cc @technetos for a last look and push the merge button :^) r? @technetos |
ecdd165
to
82ba2ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove all leads from archived teams too, please.
82ba2ac
to
bac5125
Compare
Archived teams still have the leads? |
bac5125
to
9dd166f
Compare
9dd166f
to
82c1e4b
Compare
sorry. pushed those, too |
This patch does some cleanup.
This is part of the work stemming from rust-lang/compiler-team#848
Thanks