Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove [Xml]TreeBuilderActions and [Xml]TreeBuilderStep and move their implementations into [Xml]TreeBuilder #274

Merged
merged 4 commits into from
Aug 9, 2017

Conversation

jbg
Copy link
Contributor

@jbg jbg commented Jun 1, 2017

See issue #258


This change is Reviewable

@jdm
Copy link
Member

jdm commented Jun 4, 2017

r? @nox

@jdm
Copy link
Member

jdm commented Jul 7, 2017

Review ping, @nox.

@Ygg01
Copy link
Contributor

Ygg01 commented Aug 9, 2017

Reviewed 3 of 3 files at r1, 1 of 1 files at r4.
Review status: 2 of 6 files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@jdm
Copy link
Member

jdm commented Aug 9, 2017

@bors-servo: r=Ygg01

@bors-servo
Copy link
Contributor

📌 Commit 524afe1 has been approved by Ygg01

@bors-servo
Copy link
Contributor

⌛ Testing commit 524afe1 with merge 6e5ef0d...

bors-servo pushed a commit that referenced this pull request Aug 9, 2017
Remove [Xml]TreeBuilderActions and [Xml]TreeBuilderStep and move their implementations into [Xml]TreeBuilder

See issue #258

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

☀️ Test successful - status-travis
Approved by: Ygg01
Pushing 6e5ef0d to master...

@bors-servo bors-servo merged commit 524afe1 into servo:master Aug 9, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants