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

Use less fragile and more readable numbering for ElementState. #16085

Merged
merged 1 commit into from Mar 22, 2017

Conversation

@bzbarsky
Copy link
Contributor

bzbarsky commented Mar 22, 2017


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #__ (github issue number if applicable).
  • There are tests for these changes OR
  • These changes do not require tests because it's just cleanup preliminary to another change I'm working on.

This change is Reviewable

@highfive
Copy link

highfive commented Mar 22, 2017

Heads up! This PR modifies the following files:

  • @bholley: components/style/element_state.rs
  • @emilio: components/style/element_state.rs
@highfive
Copy link

highfive commented Mar 22, 2017

warning Warning warning

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

emilio commented Mar 22, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Mar 22, 2017

📌 Commit 5415e8a has been approved by emilio

@highfive highfive assigned emilio and unassigned jdm Mar 22, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Mar 22, 2017

Testing commit 5415e8a with merge 80e914e...

bors-servo added a commit that referenced this pull request Mar 22, 2017
…emilio

Use less fragile and more readable numbering for ElementState.

<!-- Please describe your changes on the following line: -->

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because it's just cleanup preliminary to another change I'm working on.

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

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

bors-servo commented Mar 22, 2017

💔 Test failed - linux-rel-wpt

@emilio
Copy link
Member

emilio commented Mar 22, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Mar 22, 2017

Testing commit 5415e8a with merge 405ab82...

bors-servo added a commit that referenced this pull request Mar 22, 2017
…emilio

Use less fragile and more readable numbering for ElementState.

<!-- Please describe your changes on the following line: -->

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because it's just cleanup preliminary to another change I'm working on.

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

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

bors-servo commented Mar 22, 2017

☀️ 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
Approved by: emilio
Pushing 405ab82 to master...

@bors-servo bors-servo merged commit 5415e8a into servo:master Mar 22, 2017
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
@bzbarsky bzbarsky deleted the bzbarsky:better-numbering-of-pseudoclasses branch May 16, 2017
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

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