Skip to content

Bump html5ever to 0.5.5#13801

Merged
bors-servo merged 1 commit intoservo:masterfrom
nox:h5e
Oct 17, 2016
Merged

Bump html5ever to 0.5.5#13801
bors-servo merged 1 commit intoservo:masterfrom
nox:h5e

Conversation

@nox
Copy link
Copy Markdown
Contributor

@nox nox commented Oct 17, 2016 •
edited by larsbergstrom
Loading

This change is Reviewable

@highfive
Copy link
Copy Markdown

highfive commented Oct 17, 2016

Heads up! This PR modifies the following files:

  • @fitzgen: components/script/dom/servoparser/html.rs
  • @KiChjang: components/script/dom/servoparser/html.rs

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 17, 2016
@highfive
Copy link
Copy Markdown

highfive commented Oct 17, 2016

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!

@jdm
Copy link
Copy Markdown
Member

jdm commented Oct 17, 2016

@bors-servo: r+

@bors-servo
Copy link
Copy Markdown
Contributor

bors-servo commented Oct 17, 2016

📌 Commit b6d6b3b has been approved by jdm

@highfive highfive assigned jdm and unassigned pcwalton Oct 17, 2016
@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Oct 17, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

bors-servo commented Oct 17, 2016

⌛ Testing commit b6d6b3b with merge cbe2f7e...

bors-servo pushed a commit that referenced this pull request Oct 17, 2016
Auto merge of #13801 - nox:h5e, r=jdm
cbe2f7e 
Bump html5ever to 0.5.5

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13801)
<!-- Reviewable:end -->
@bors-servo bors-servo mentioned this pull request Oct 17, 2016
Merged
@bors-servo
Copy link
Copy Markdown
Contributor

bors-servo commented Oct 17, 2016

☀️ Test successful - arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-dev

@bors-servo bors-servo merged commit b6d6b3b into servo:master Oct 17, 2016
@highfive highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 17, 2016
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

Development

Successfully merging this pull request may close these issues.

5 participants

@nox @highfive @jdm @bors-servo @pcwalton