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

Add GitHub info for wg-nll #736

Merged
merged 1 commit into from Apr 29, 2022
Merged

Add GitHub info for wg-nll #736

merged 1 commit into from Apr 29, 2022

Conversation

rylev
Copy link
Member

@rylev rylev commented Apr 13, 2022

The types team is the likely successor of the work.

Note: I also added a [[github]] annotation to the wg defintion in the unlikely case that the wg is revived, since until now the permissions have been manually curated on GitHub.

cc @nikomatsakis @pnkfelix

@jackh726
Copy link
Member

I think it's probably just worth waiting until NLL stabilization. That's hopefully coming soon™

While I think most of the members here aren't active much anyways, the group as an entity is good to help organize the final stabilization push. (I'm mostly thinking the wg-nll zulip stream. The formal working group might very well be overkill.)

@rylev
Copy link
Member Author

rylev commented Apr 14, 2022

I'll let @nikomatsakis weigh in here, but I am of the opinion that when in doubt we should keep the wg around. However, I believe Niko was keen on seeing this responsibility moved to the new types team.

@jackh726
Copy link
Member

Oh this definitely would fall under the types team. But I think this would still exist, either in the form of an initiative or a working group.

@rylev
Copy link
Member Author

rylev commented Apr 14, 2022

@jackh726 that sounds reasonable. One additional thing to consider is that we will also be syncing Zulip user groups through the team repo (see rust-lang/sync-team#3) so if we want @wg-nll to work on Zulip, we should still keep it around.

@rylev rylev changed the title Archive wg-nll as it is no longer active Add GitHub info for wg-nll Apr 15, 2022
@rylev
Copy link
Member Author

rylev commented Apr 15, 2022

Ok I shifted this to just add the GitHub info to the wg definition since that should be without controversy. We can still archive at a later point.

@Mark-Simulacrum
Copy link
Member

This will create a duplicate team with wg-compiler-nll, which has many more members. Happy to merge, but we should probably have a follow-up of deduplicating here one way or another. I'm not sure if there's repositories wg-compiler-nll has access to.

@rylev
Copy link
Member Author

rylev commented Apr 27, 2022

@Mark-Simulacrum indeed. The plan is to remove the wg-compiler-nll group in favor of wg-nll. Before removing it, I would check with the members who are not in the working group but are in the old user group to ensure we reach consistency.

@rylev rylev merged commit 4b61143 into rust-lang:master Apr 29, 2022
@rylev rylev deleted the archive-wg-nll branch April 29, 2022 09:10
@rylev rylev mentioned this pull request Sep 28, 2022
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.

None yet

3 participants