Skip to content

/ saltfs

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

Allow birtles to use try and land PRs #668

Merged
merged 2 commits into from May 17, 2017
Merged

Allow birtles to use try and land PRs #668

merged 2 commits into from May 17, 2017

Conversation

@birtles
Copy link
Contributor

birtles commented May 15, 2017 •
edited by larsbergstrom

This change is Reviewable
@birtles
Allow birtles to use try and land PRs
Loading status checks…
b672ecc
@highfive
Copy link

highfive commented May 15, 2017

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @aneeshusa (or someone else) soon.
@birtles
Copy link
Contributor Author

birtles commented May 15, 2017

Hi, I'd like to get permission to land PRs so I don't have to keep asking for delegate+ for Bugzilla-reviewed patches.
@aneeshusa
Copy link
Member

aneeshusa commented May 17, 2017

r? @metajack or @larsbergstrom or @jdm
@highfive highfive assigned metajack and unassigned aneeshusa May 17, 2017
@aneeshusa
Copy link
Member

aneeshusa commented May 17, 2017

@birtles Since you're updating the operator list, can you please fix the alphabetization (broken by #667) as well?
@birtles
Fix alphabetization of operator list
Loading status checks…
27ab467
@birtles
Copy link
Contributor Author

birtles commented May 17, 2017

Yes, of course. Fixed.
@jdm
Copy link
Member

jdm commented May 17, 2017

@bors-servo: r+
@bors-servo
Copy link
Contributor

bors-servo commented May 17, 2017

📌 Commit 27ab467 has been approved by jdm
@highfive highfive assigned jdm and unassigned metajack May 17, 2017
@bors-servo
Copy link
Contributor

bors-servo commented May 17, 2017

⌛ Testing commit 27ab467 with merge 5d3e42b...
bors-servo added a commit that referenced this pull request May 17, 2017
@bors-servo
Auto merge of #668 - birtles:patch-1, r=jdm
Loading status checks…
5d3e42b 
Allow birtles to use try and land PRs

<!-- 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/668)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 17, 2017

☀️ Test successful - status-travis
Approved by: jdm
Pushing 5d3e42b to master...
@bors-servo bors-servo merged commit 27ab467 into servo:master May 17, 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
@birtles birtles mentioned this pull request May 23, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jdm jdm

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
@birtles @highfive @aneeshusa @jdm @bors-servo @metajack
You can’t perform that action at this time.