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 Behnam Esfahbod as a reviewer for unicode-bidi #670

Merged
merged 1 commit into from May 17, 2017
Merged

Conversation

@mbrubeck
Copy link
Contributor

mbrubeck commented May 17, 2017

@behnam is currently the most active developer of the unicode-bidi crate and has now written or rewritten large portions of the code.


This change is Reviewable

@behnam is currently the most active developer of the unicode-bidi crate and has now written or rewritten large portions of the code.
@Manishearth
Copy link
Member

Manishearth commented May 17, 2017

r+

@Manishearth
Copy link
Member

Manishearth commented May 17, 2017

wait i forget what's our policy on adding reviewers again, do we need core approval?

@jdm
Copy link
Member

jdm commented May 17, 2017

For our smaller repositories the process can be a bit more fluid. This case is interesting because it's a crate that until now has had very little active development, so we can't really link to any existing reviews. I think this makes sense, however.
@bors-servo: r+

@bors-servo
Copy link
Contributor

bors-servo commented May 17, 2017

📌 Commit eb4abda has been approved by jdm

@highfive highfive assigned jdm and unassigned aneeshusa May 17, 2017
@bors-servo
Copy link
Contributor

bors-servo commented May 17, 2017

Testing commit eb4abda with merge e7fd6b7...

bors-servo added a commit that referenced this pull request May 17, 2017
Add Behnam Esfahbod as a reviewer for unicode-bidi

@behnam is currently the most active developer of the unicode-bidi crate and has now written or rewritten large portions of the code.

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

bors-servo commented May 17, 2017

☀️ Test successful - status-travis
Approved by: jdm
Pushing e7fd6b7 to master...

@bors-servo bors-servo merged commit eb4abda into master May 17, 2017
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the unicode-bidi branch Dec 20, 2017
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.