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

Fix a panic when using layout 2020 #23920

Merged
merged 1 commit into from Aug 6, 2019
Merged

Fix a panic when using layout 2020 #23920

merged 1 commit into from Aug 6, 2019

Conversation

@nox
Copy link
Member

nox commented Aug 6, 2019

This change is Reviewable

@highfive
Copy link

highfive commented Aug 6, 2019

Heads up! This PR modifies the following files:

  • @emilio: components/style/properties/properties.mako.rs
@highfive
Copy link

highfive commented Aug 6, 2019

warning Warning warning

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

SimonSapin commented Aug 6, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

📌 Commit 71c5815 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

Testing commit 71c5815 with merge 01de366...

bors-servo added a commit that referenced this pull request Aug 6, 2019
Fix a panic when using layout 2020

<!-- 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/23920)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented Aug 6, 2019

That failure is #21067, but test-tidy fails:

�[94m./components/style/properties/properties.mako.rs�[0m:�[93m508�[0m: �[91mLine is longer than 120 characters�[0m
@jdm jdm added the S-fails-tidy label Aug 6, 2019
@nox nox force-pushed the fix-style-2020 branch from 71c5815 to 8e6591b Aug 6, 2019
@nox
Copy link
Member Author

nox commented Aug 6, 2019

Oops, fixed.

@bors-servo r=SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

📌 Commit 8e6591b has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

Testing commit 8e6591b with merge ecc2b7f...

bors-servo added a commit that referenced this pull request Aug 6, 2019
Fix a panic when using layout 2020

<!-- 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/23920)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member

jdm commented Aug 6, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

Testing commit 8e6591b with merge 0368cd8...

bors-servo added a commit that referenced this pull request Aug 6, 2019
Fix a panic when using layout 2020

<!-- 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/23920)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 6, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: SimonSapin
Pushing 0368cd8 to master...

@bors-servo bors-servo merged commit 8e6591b into master Aug 6, 2019
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
Taskcluster (pull_request) TaskGroup: success
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the fix-style-2020 branch Sep 30, 2019
@SimonSapin SimonSapin added this to Done / resolved in Layout 2020 Dec 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Layout 2020
  
Merged / resolved
Linked issues

Successfully merging this pull request may close these issues.

None yet

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