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

update and archive project-thir-unsafeck #1422

Merged
merged 3 commits into from
Apr 5, 2024
Merged

Conversation

lcnr
Copy link
Contributor

@lcnr lcnr commented Apr 5, 2024

THIR unsafeck is now stable since 1.77 (rust-lang/rust#117673) and MIR unsafeck has since been removed (rust-lang/rust#123322) 🎉 I think we should therefore archive the working group.

It was also outdated. I added @LeSeulArtichaut to alumni, which afaict was not possible when they left the project in #621. I also added @matthewjasper as a member as they've been doing most of the work towards actually getting it stabilized.

Unsure what to do with the afaict dead zulip stream (https://rust-lang.zulipchat.com/#narrow/stream/278509-project-thir-unsafeck). Ideally it should be archived and locked.

cc @nikomatsakis

@Kobzol
Copy link
Contributor

Kobzol commented Apr 5, 2024

Can you please also add a commit that moves the repos/rust-lang/project-thir-unsafeck directory under repos/archive/rust-lang/project-thir-unsafeck? And also remove the access for the project-thir-unsafeck project group from the repo (just keep empty [access.teams]). That will archive the repo and should fix the CI error.

@Kobzol
Copy link
Contributor

Kobzol commented Apr 5, 2024

Ah, one more error, all members of archived teams must be alumni.

@lcnr
Copy link
Contributor Author

lcnr commented Apr 5, 2024

Should have maybe looked at the CI more quickly myself to save you some work 😊 thanks ❤️

@rylev rylev merged commit f8381d6 into rust-lang:master Apr 5, 2024
1 check passed
@lcnr lcnr deleted the thir-unsafe-ck branch April 5, 2024 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants