-
-
Notifications
You must be signed in to change notification settings - Fork 553
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
docs: add AndEsterson as a contributor for infra #4666
Conversation
The "style" status check shouldn't really be set to required because commits that skip all CI jobs (such as this one) won't be possible to merge otherwise. |
Someone with administrator permissions will need to merge this 🙂 |
Oh yes. @kratman could maybe help with this as Ferran is vacationing. |
Style checks should be required. We should require as many checks as possible. I will see if I can find a work around for this bot |
I put up a fix PR for this, we might need to re-create this PR to fully tests it though |
We will have to wait for another automated PR to test the infrastructure - see #4661 (comment) |
Adds @AndEsterson as a contributor for infra.
This was requested by Saransh-cpp in this comment
[skip ci]