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 Caleb as co-lead of rustfmt #551

Merged
merged 1 commit into from
Mar 17, 2021
Merged

Conversation

Manishearth
Copy link
Member

cc @calebcartwright @topecongiro

We've been unable to reach @topecongiro for a while, and it would be nice if Caleb had all the perms so that he can make progress on rust-lang/rust#82385 . He's been doing a lot of the rustfmt work for quite a while now.

@topecongiro, let me know when you get back and we can discuss this.

r? @rust-lang/devtools

@kinnison
Copy link
Contributor

Caleb will also need to have an email address registered, bsaed on the CI failures.

killercup
killercup previously approved these changes Mar 16, 2021
Xanewok
Xanewok previously approved these changes Mar 16, 2021
@calebcartwright
Copy link
Member

Caleb will also need to have an email address registered, bsaed on the CI failures.

Yup, Manish mentioned this a while back but I never got around to it. Opened #552

@Manishearth
Copy link
Member Author

cc @Mark-Simulacrum can you rubber stamp?

@Manishearth Manishearth reopened this Mar 16, 2021
@Mark-Simulacrum
Copy link
Member

It looks like you'll need to rebase or otherwise get the commit bumped for CI to pass here (I guess it runs on the push so open/close didn't help). Otherwise this looks good to me.

@Manishearth
Copy link
Member Author

Done

@Mark-Simulacrum Mark-Simulacrum merged commit 35974d8 into master Mar 17, 2021
@Mark-Simulacrum Mark-Simulacrum deleted the Manishearth-patch-2 branch March 17, 2021 02:21
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.

6 participants