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

homu: Let hiro land his PRs reviewed in bugzilla. #652

Merged
merged 1 commit into from May 2, 2017

Conversation

@emilio
Copy link
Member

emilio commented May 1, 2017

He's been doing a lot of stylo work recently, and I got tired of delegating all
his patches :).


This change is Reviewable

He's been doing a lot of stylo work recently, and I got tired of delegating all
his patches :).
@emilio
Copy link
Member Author

emilio commented May 1, 2017

@highfive highfive assigned larsbergstrom and unassigned aneeshusa May 1, 2017
@larsbergstrom
Copy link
Contributor

larsbergstrom commented May 1, 2017

@emilio What does this specifically mean here? You're saying that he gets review for the Servo bits in some other place and needs the ability to r=emilio them here without having one of you do it?

Sorry for being a bit dense here - the text of this PR and the commit are asking for him to get reviewer privs, which requires review of some of their reviews, and to be allowed to self-r+, which is not allowed in Servo, so I'm a little confused :-)

@hiikezoe
Copy link

hiikezoe commented May 2, 2017

@larsbergstrom It's enough to me that I have the permission to do delegate+ for PRs that patches have been already reviewed in bugzilla.

@emilio
Copy link
Member Author

emilio commented May 2, 2017

Right, so this PR puts him in the operator list, in order to land his reviewed-in-bugzilla PRs.

Sorry, wasn't trying to imply self-r+.

@emilio emilio changed the title homu: Let hiro land his own PRs. homu: Let hiro land his PRs reviewed in bugzilla. May 2, 2017
@larsbergstrom
Copy link
Contributor

larsbergstrom commented May 2, 2017

Thanks, that will be fine, then :-)

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented May 2, 2017

📌 Commit 26e7ba8 has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented May 2, 2017

Testing commit 26e7ba8 with merge 877276a...

bors-servo added a commit that referenced this pull request May 2, 2017
homu: Let hiro land his PRs reviewed in bugzilla.

He's been doing a lot of stylo work recently, and I got tired of delegating all
his patches :).

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/652)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 2, 2017

☀️ Test successful - status-travis
Approved by: larsbergstrom
Pushing 877276a to master...

@bors-servo bors-servo merged commit 26e7ba8 into servo:master May 2, 2017
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
@aneeshusa
Copy link
Member

aneeshusa commented May 3, 2017

FYI, I have deployed this so bors-servo should now recognize your new permissions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.