Skip to content

Update to unicode-bidi 0.2.1#6998

Merged
bors-servo merged 1 commit intoservo:masterfrom
mbrubeck:upgrade
Aug 6, 2015
Merged

Update to unicode-bidi 0.2.1#6998
bors-servo merged 1 commit intoservo:masterfrom
mbrubeck:upgrade

Conversation

@mbrubeck
Copy link
Copy Markdown
Contributor

@mbrubeck mbrubeck commented Aug 5, 2015

Also includes some missing changes from #6416.

Review on Reviewable

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Aug 5, 2015
@Manishearth
Copy link
Copy Markdown
Member

@bors r+

@mbrubeck
Copy link
Copy Markdown
Contributor Author

mbrubeck commented Aug 5, 2015

Just a moment, this is missing a change...

@mbrubeck
Copy link
Copy Markdown
Contributor Author

mbrubeck commented Aug 5, 2015

@bors-servo r=Manishearth

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 66379bc has been approved by Manishearth

@jdm jdm 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 Aug 5, 2015
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 66379bc with merge e302474...

bors-servo pushed a commit that referenced this pull request Aug 6, 2015
@bors-servo
Copy link
Copy Markdown
Contributor

💔 Test failed - linux2

@jdm
Copy link
Copy Markdown
Member

jdm commented Aug 6, 2015

/css21_dev/html4/first-letter-dynamic-002.htm
---------------------------------------------
PASS expected FAIL [Parent]

@jdm jdm added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Aug 6, 2015
@highfive highfive added S-awaiting-review There is new code that needs to be reviewed. and removed S-tests-failed The changes caused existing tests to fail. labels Aug 6, 2015
@mbrubeck
Copy link
Copy Markdown
Contributor Author

mbrubeck commented Aug 6, 2015

@bors-servo r=Manishearth

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 73ce232 has been approved by Manishearth

@jdm jdm 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 Aug 6, 2015
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 73ce232 with merge d2ba81e...

bors-servo pushed a commit that referenced this pull request Aug 6, 2015
@bors-servo
Copy link
Copy Markdown
Contributor

☀️ Test successful - android, gonk, linux1, linux2, mac1, mac2, mac3

@bors-servo bors-servo merged commit 73ce232 into servo:master Aug 6, 2015
@mbrubeck mbrubeck deleted the upgrade branch May 11, 2016 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants