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 new crate maintainers team. #861

Merged
merged 1 commit into from
Oct 21, 2022
Merged

Conversation

m-ou-se
Copy link
Member

@m-ou-se m-ou-se commented Oct 5, 2022

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks right to me 👍

@rylev
Copy link
Member

rylev commented Oct 6, 2022

Question to naming: the name "crates" might be a bit confusing when seen without the context of it being a libs subteam. I wonder if there's another name that's just as succinct but less prone to being misunderstood.

Also, I missed any discussion about why this changed was deemed necessary. If there's any link to such discussion, I would personally find it useful (also for those who stumble upon this PR in the future).

@JohnTitor
Copy link
Member

JohnTitor commented Oct 9, 2022

Question to naming: the name "crates" might be a bit confusing when seen without the context of it being a libs subteam. I wonder if there's another name that's just as succinct but less prone to being misunderstood.

Also, I missed any discussion about why this changed was deemed necessary. If there's any link to such discussion, I would personally find it useful (also for those who stumble upon this PR in the future).

This meeting has related discussion: https://rust-lang.zulipchat.com/#narrow/stream/259402-t-libs.2Fmeetings/topic/Meeting.202022-09-14/near/298789218
And meetings on other dates also have some more discussion, you could search "crate-maintainers" or something else on Zulip.

@m-ou-se
Copy link
Member Author

m-ou-se commented Oct 11, 2022

Question to naming: the name "crates" might be a bit confusing when seen without the context of it being a libs subteam. I wonder if there's another name that's just as succinct but less prone to being misunderstood.

"team responsible for maintaining the crates that are maintained by the rust team but are not the standard library" is a bit long, but I'm not sure what's the best way to shorten it. Suggestions welcome. :)

@m-ou-se m-ou-se changed the title [do not merge yet] Add new crate maintainers team. Add new crate maintainers team. Oct 20, 2022
@m-ou-se m-ou-se marked this pull request as ready for review October 20, 2022 07:34
@m-ou-se
Copy link
Member Author

m-ou-se commented Oct 20, 2022

@rylev This is ready for merging, except for the team name bikeshed.

crate-maintainers? libs-crate-maintainers? official-but-not-quite-standard-crates-maintainers? log-libc-compiler-builtins-etc-etc-etc-maintainers? naming-is-hard? Very open for better suggestions. ^^'

@rylev
Copy link
Member

rylev commented Oct 20, 2022

I think crate-maintainers is probably fine. I'll leave this open for a little while longer to see if any other opinions come in for names, but then we can merge this.

@weihanglo
Copy link
Member

The description of Library team says

Managing and maintaining the Rust standard library and official rust-lang crates.

Since it is a subteam of libs team, it is very clear to me that it maintains official rust-lang crates. At least as an outsider :)

Though I am more curious about how one knows which crate is maintained by this team?

Copy link
Member

@rylev rylev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll merge this now.

@m-ou-se does it make sense to add a Zulip user group for pinging these folks or is it too much of a catch-all for that to be useful?

@rylev rylev merged commit 3661d5a into rust-lang:master Oct 21, 2022
@m-ou-se m-ou-se deleted the crate-maintainers branch October 21, 2022 08:50
@m-ou-se
Copy link
Member Author

m-ou-se commented Oct 21, 2022

@rylev Yes. Is that something that can happen automatically now? Or do we need to manually create that in Zulip?

Edit: PR here: #866

@m-ou-se
Copy link
Member Author

m-ou-se commented Oct 21, 2022

@pietroalbini can you give this new crate-maintainers group GitHub write and Crates.io publish access to libc, compiler-builtins, libm, log, libz-sys, cfg-if, cc, cmake, pkg-config, glob, flate2, socket2, getopts, and backtrace-rs?

@rylev
Copy link
Member

rylev commented Oct 21, 2022

Oh I would love to do this through the repo configuration option in the team repo (since we have to do that eventually anyhow). How urgent is this? (i.e., if it takes a day or two is that fine?)

@m-ou-se
Copy link
Member Author

m-ou-se commented Oct 21, 2022

@rylev It'd be nice to fix it for libc right now, because the old libc team was removed by this PR. For the rest, a few days delay is fine. :)

@rylev
Copy link
Member

rylev commented Oct 21, 2022

@m-ou-se FYI: sync-team does not yet delete GitHub teams that are defined on GitHub but not in the team repo (though I'd like to add that very soon). In any case, I've added the new team to the libc repo anyway.

@thomcc
Copy link
Member

thomcc commented Oct 25, 2022

In rust-lang/cc-rs#661 (comment) I realized I don't seem to have the ability to merge PRs in cc, should I? I do for some of the other crates.

(No worries if you just haven't gotten to it, I'm mostly making sure this isn't a case of some tool failing)

@rylev
Copy link
Member

rylev commented Oct 25, 2022

@thomcc I've added crate-maintainers to the cc-rs repo so you should be good to go.

@thomcc
Copy link
Member

thomcc commented Oct 26, 2022

@rylev I promised I'd cut a release of cmake-rs in rust-lang/cmake-rs#162 (comment), but after doing a review I realize it doesn't seem like I have access to that either.

Can you double-check versus the list @m-ou-se gave in this comment?

@thomcc
Copy link
Member

thomcc commented Oct 26, 2022

(Oh, sorry, this is being done in #879, my bad!)

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.

5 participants