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
Handle length parsing mode for other types too #16596
Conversation
MozReview-Commit-ID: CIJQmx84QB2
r=me @bors-servo delegate+ |
✌️ @Manishearth can now approve this pull request |
Hm, looks like it doesn't affect try much, those tests have other reasons to fail too. @bors-servo r=heycam |
📌 Commit 1639096 has been approved by |
Handle length parsing mode for other types too All length types should handle this. Seems to block a couple tests. try at https://treeherder.mozilla.org/#/jobs?repo=try&revision=b1eb64509e65f9dc548d361ac2ae00ce000c9d7b <!-- 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/16596) <!-- Reviewable:end -->
💔 Test failed - linux-rel-css |
⚡ Previous build results for android, arm32, arm64, linux-dev, mac-dev-unit, mac-rel-wpt1, windows-msvc-dev are reusable. Rebuilding only linux-rel-css, linux-rel-wpt, mac-rel-css, mac-rel-wpt2... |
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-msvc-dev |
Thanks for covering these other cases! 😁 |
All length types should handle this. Seems to block a couple tests.
try at https://treeherder.mozilla.org/#/jobs?repo=try&revision=b1eb64509e65f9dc548d361ac2ae00ce000c9d7b
This change is