Skip to content

Remove individual access to the ssh2-rs repository - #2919

Draft
Kobzol wants to merge 2 commits into
rust-lang:mainfrom
Kobzol:ssh2-access
Draft

Kobzol wants to merge 2 commits into
rust-lang:mainfrom
Kobzol:ssh2-access

Conversation

@Kobzol

@Kobzol Kobzol commented Nov 7, 2026 •

Copy link
Copy Markdown
Member

We want to get rid of all access of individuals to repositories, where applicable. This PR can be merged once we move the crates of this repo to Trusted Publishing (#2906)

CC @crisidev @wez @yodaldevoid Let me know if you're fine with losing access to the repo, or if you are still maintaining it.

@Kobzol
Kobzol marked this pull request as draft November 7, 2026 15:48
@wez

wez commented Nov 7, 2026

Copy link
Copy Markdown

I don't need any special direct access any longer, and can be removed. Thanks :)

@crisidev

crisidev commented Nov 7, 2026

Copy link
Copy Markdown

Same, I don't need individual access. Thanks for doing this!

@Kobzol

Kobzol commented Nov 7, 2026

Copy link
Copy Markdown
Member Author

Ah, I would have to remove your accounts from team entirely, since this was your only entry in team. I can either do that, or sort of artificially make you an alumni of crate-maintainers. @clarfonthey What do you think?

@clarfonthey

Copy link
Copy Markdown
Contributor

Yeah, I think just shoving all individuals on repos that maintained crates into crate-maintainers alums makes sense, even if said crates were never under that team. The alternative would be marking them as cargo alums, though, so either/or imho.

I've been thinking about doing a broader push to centralise all published org crates that aren't just, published versions of The Thing The Team Works On (e.g. rustc_* crates or cargo_* crates) into the crate-maintainers team, but that obviously would be a bigger proposal which requires buy-in from everyone involved.

For this case I don't think it's too big a deal though, since being an alum doesn't grant you any permissions other than being recognised.

@Kobzol

Kobzol commented Nov 7, 2026

Copy link
Copy Markdown
Member Author

Makes sense. Added the users to crate-maintainers alumni.

@yodaldevoid

yodaldevoid commented Nov 7, 2026 •

Copy link
Copy Markdown

I still check in on the repo regularly, but am more than fine loosing individual access.

@github-actions

github-actions Bot commented Nov 7, 2026

Copy link
Copy Markdown
Dry-run check results

[WARN  rust_team::sync] sync-team is running in dry mode, no changes will be applied.
[INFO  rust_team::sync] synchronizing crates-io
[INFO  rust_team::sync] synchronizing github
[INFO  rust_team::sync] Rate limits before: REST 14012/15000 remaining, resets at 2026-11-07 18:52:49 UTC, GraphQL 12410/12500 remaining, resets at 2026-11-07 18:52:41 UTC
[INFO  rust_team::sync] 💻 Repo Diffs:
    📝 Editing repo 'rust-lang/ssh2-rs':
      Permission Changes:
        Removing user 'crisidev''s write permission 
        Removing user 'yodaldevoid''s write permission 
        Removing user 'wez''s write permission 

[INFO rust_team::sync] Rate limits after: REST 13024/15000 remaining, resets at 2026-11-07 18:52:49 UTC, GraphQL 12316/12500 remaining, resets at 2026-11-07 18:52:41 UTC
[INFO rust_team::sync] Consumed 988 REST calls and 94 GraphQL calls

This branch has not been deployed

No deployments
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