-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Enable cors and redirect fetch tests. #14070
Conversation
Heads up! This PR modifies the following files:
|
@bors-servo: try |
[Include mode: 1 cookie] | ||
[Include mode: local cookies are not sent with remote request] | ||
expected: FAIL | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this doesn't seem good
(yes, we already failed here before this PR, but we should fix this)
r=me if try doesn't turn up anything other than new passes |
💔 Test failed - linux-rel-wpt |
@bors-servo: try |
Enable cors and redirect fetch tests. There have been a bunch of changes to the fetch code since #13441 was filed, and the results seem more stable now. <!-- 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/14070) <!-- Reviewable:end -->
💥 Test timed out |
@bors-servo: retry |
Enable cors and redirect fetch tests. There have been a bunch of changes to the fetch code since #13441 was filed, and the results seem more stable now. <!-- 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/14070) <!-- Reviewable:end -->
@bors-servo: retry |
Enable cors and redirect fetch tests. There have been a bunch of changes to the fetch code since #13441 was filed, and the results seem more stable now. <!-- 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/14070) <!-- Reviewable:end -->
☀️ 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: r=Manishearth |
📌 Commit a2468a4 has been approved by |
@bors-servo try- retry clean |
Enable cors and redirect fetch tests. There have been a bunch of changes to the fetch code since #13441 was filed, and the results seem more stable now. <!-- 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/14070) <!-- Reviewable:end -->
💔 Test failed - linux-rel-wpt |
|
🔒 Merge conflict |
☔ The latest upstream changes (presumably #13918) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors-servo: r=Manishearth |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit a2468a4 has been approved by |
@bors-servo: retry |
@bors-servo: clean r=Manishearth |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit a2468a4 has been approved by |
Closed in favour of #14116. |
There have been a bunch of changes to the fetch code since #13441 was filed, and the results seem more stable now.
This change is