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 Ygg01 to html5ever reviewers #926

Merged
merged 1 commit into from Dec 17, 2018
Merged

Add Ygg01 to html5ever reviewers #926

merged 1 commit into from Dec 17, 2018

Conversation

@jdm
Copy link
Member

jdm commented Dec 13, 2018

@Ygg01 has been a long-time contributor to the repository and knows the code well.


This change is Reviewable

@emilio
Copy link
Member

emilio commented Dec 17, 2018

@bors-servo r+

Congratulations / thanks for all your work @Ygg01!

@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2018

📌 Commit 8e52404 has been approved by emilio

@bors-servo
Copy link
Contributor

bors-servo commented Dec 17, 2018

Testing commit 8e52404 with merge cca6ebe...

bors-servo added a commit that referenced this pull request Dec 17, 2018
Add Ygg01 to html5ever reviewers

@Ygg01 has been a long-time contributor to the repository and knows the code well.

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

bors-servo commented Dec 17, 2018

☀️ Test successful - status-travis
Approved by: emilio
Pushing cca6ebe to master...

@bors-servo bors-servo merged commit 8e52404 into master Dec 17, 2018
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@Ygg01
Copy link

Ygg01 commented Dec 18, 2018

@emilio Thank you.

With that said UNLIMITED POWER

@Ygg01
Copy link

Ygg01 commented Dec 18, 2018

Btw is there a way for me to see commands are and what they do?

@jdm
Copy link
Member Author

jdm commented Dec 18, 2018

Scroll to the bottom of https://build.servo.org/homu/.

@jdm jdm removed the S-needs-deploy label Jan 11, 2019
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

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