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

Include square brackets for IPv6 addresses in Document::domain. #10287

Merged
merged 1 commit into from Mar 31, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Mar 30, 2016

The specification changed out from under us.

It's unfortunately impossible to write a test for this right now.


This change is Reviewable

The specification changed out from under us.

It's unfortunately impossible to write a test for this right now.
@highfive
Copy link

highfive commented Mar 30, 2016

Heads up! This PR modifies the following files:

  • @KiChjang: components/script/dom/document.rs
@highfive
Copy link

highfive commented Mar 30, 2016

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!
@jdm
Copy link
Member

jdm commented Mar 30, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 30, 2016

📌 Commit 231208a has been approved by jdm

@jdm jdm assigned jdm and unassigned pcwalton Mar 30, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Mar 30, 2016

Testing commit 231208a with merge 19c7fa8...

bors-servo added a commit that referenced this pull request Mar 30, 2016
Include square brackets for IPv6 addresses in Document::domain.

The specification changed out from under us.

It's unfortunately impossible to write a test for this right now.

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

larsbergstrom commented Mar 30, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 30, 2016

Testing commit 231208a with merge ff42f24...

bors-servo added a commit that referenced this pull request Mar 30, 2016
Include square brackets for IPv6 addresses in Document::domain.

The specification changed out from under us.

It's unfortunately impossible to write a test for this right now.

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

bors-servo commented Mar 30, 2016

💔 Test failed - status-appveyor

@jdm
Copy link
Member

jdm commented Mar 30, 2016

@bors-servo: retry

  • appveyor timeout exceeded due to unit tests
@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2016

Testing commit 231208a with merge af06d32...

bors-servo added a commit that referenced this pull request Mar 31, 2016
Include square brackets for IPv6 addresses in Document::domain.

The specification changed out from under us.

It's unfortunately impossible to write a test for this right now.

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

bors-servo commented Mar 31, 2016

@bors-servo bors-servo merged commit 231208a into servo:master Mar 31, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@bors-servo
Copy link
Contributor

bors-servo commented Mar 31, 2016

💥 Test timed out

@Ms2ger Ms2ger deleted the Ms2ger:domain branch Mar 31, 2016
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

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