Skip to content

Move tests of idna to serde-json#518

Merged
bors-servo merged 1 commit intoservo:masterfrom
est31:idna_serde
Jul 18, 2019
Merged

Move tests of idna to serde-json#518
bors-servo merged 1 commit intoservo:masterfrom
est31:idna_serde

Conversation

@est31
Copy link
Copy Markdown
Contributor

@est31 est31 commented Jul 17, 2019

Part of #506


This change is Reviewable

@est31 est31 mentioned this pull request Jul 17, 2019
3 tasks
@est31 est31 changed the title Move tests of idna to rustc-serialize Move tests of idna to serde-json Jul 17, 2019
@bors-servo
Copy link
Copy Markdown
Contributor

☔ The latest upstream changes (presumably #517) made this pull request unmergeable. Please resolve the merge conflicts.

@nox
Copy link
Copy Markdown
Contributor

nox commented Jul 18, 2019

Cool.

@bors-servo r+

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 297606c has been approved by nox

@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 297606c with merge 3c71cac...

bors-servo pushed a commit that referenced this pull request Jul 18, 2019
Move tests of idna to serde-json

Part of #506

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

☀️ Test successful - checks-travis
Approved by: nox
Pushing 3c71cac to master...

@bors-servo bors-servo merged commit 297606c into servo:master Jul 18, 2019
takumi-earth pushed a commit to earthlings-dev/rust-url that referenced this pull request Jan 27, 2026
Move tests of idna to serde-json

Part of servo#506

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/518)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants