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

Remove unused imports. #12903

Merged
merged 1 commit into from Aug 19, 2016
Merged

Remove unused imports. #12903

merged 1 commit into from Aug 19, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Aug 17, 2016

This change is Reviewable

@highfive
Copy link

highfive commented Aug 17, 2016

Heads up! This PR modifies the following files:

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

highfive commented Aug 17, 2016

warning Warning warning

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

wafflespeanut commented Aug 17, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

📌 Commit e749201 has been approved by Wafflespeanut

@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

Testing commit e749201 with merge 404d4da...

bors-servo added a commit that referenced this pull request Aug 17, 2016
Remove unused imports.

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

bors-servo commented Aug 17, 2016

💔 Test failed - mac-rel-css

@highfive
Copy link

highfive commented Aug 17, 2016

  ▶ TIMEOUT [expected FAIL] /css-flexbox-1_dev/html/flexbox-basic-iframe-vert-001.htm
  │ 
  └ Shutting down the Constellation after generating an output file or exit flag specified
@Ms2ger
Copy link
Contributor Author

Ms2ger commented Aug 17, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

Previous build results for arm32, arm64, linux-dev, mac-dev-unit, mac-rel-wpt, windows-dev are reusable. Rebuilding only linux-rel, mac-rel-css...

@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

💔 Test failed - linux-rel

@highfive
Copy link

highfive commented Aug 17, 2016

  ▶ FAIL [expected PASS] /css-transforms-1_dev/html/perspective-origin-005.htm
  └   → /css-transforms-1_dev/html/perspective-origin-005.htm 719003ff0f2c5efe0b678a757a454870ca362e2c
/css-transforms-1_dev/html/reference/ref-filled-green-100px-square.htm d4aa213e3e31e41d0d8e84aec79e94f860f27178
Testing 719003ff0f2c5efe0b678a757a454870ca362e2c == d4aa213e3e31e41d0d8e84aec79e94f860f27178
@KiChjang
Copy link
Member

KiChjang commented Aug 17, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 17, 2016

Testing commit e749201 with merge 7efc0e1...

bors-servo added a commit that referenced this pull request Aug 17, 2016
Remove unused imports.

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

bors-servo commented Aug 17, 2016

💔 Test failed - linux-dev

@bors-servo
Copy link
Contributor

bors-servo commented Aug 18, 2016

The latest upstream changes (presumably #12668) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 18, 2016

💔 Test failed - linux-rel

@highfive
Copy link

highfive commented Aug 18, 2016

  ▶ FAIL [expected PASS] /css-transforms-1_dev/html/perspective-origin-001.htm
  └   → /css-transforms-1_dev/html/perspective-origin-001.htm 719003ff0f2c5efe0b678a757a454870ca362e2c
/css-transforms-1_dev/html/reference/ref-filled-green-100px-square.htm d4aa213e3e31e41d0d8e84aec79e94f860f27178
Testing 719003ff0f2c5efe0b678a757a454870ca362e2c == d4aa213e3e31e41d0d8e84aec79e94f860f27178
@wafflespeanut
Copy link
Member

wafflespeanut commented Aug 19, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

Testing commit 4ede9f6 with merge 3306e5c...

bors-servo added a commit that referenced this pull request Aug 19, 2016
Remove unused imports.

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

bors-servo commented Aug 19, 2016

💔 Test failed - linux-rel

@highfive
Copy link

highfive commented Aug 19, 2016

  ▶ FAIL [expected PASS] /css-transforms-1_dev/html/transform-table-007.htm
  └   → /css-transforms-1_dev/html/transform-table-007.htm a5c014b20ef1363bea6f24eda28c7efb7c45698a
/css-transforms-1_dev/html/reference/transform-blank-ref.htm fa6407b1acbbfea27e27061e7d1bdeca98e4a728
Testing a5c014b20ef1363bea6f24eda28c7efb7c45698a == fa6407b1acbbfea27e27061e7d1bdeca98e4a728
@wafflespeanut
Copy link
Member

wafflespeanut commented Aug 19, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

Previous build results for arm32, arm64, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev are reusable. Rebuilding only linux-rel...

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

The build was interrupted to prioritize another pull request.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

Previous build results for arm32, arm64, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev are reusable. Rebuilding only linux-rel...

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

The build was interrupted to prioritize another pull request.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

Previous build results for arm32, arm64, linux-dev, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev are reusable. Rebuilding only linux-rel...

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

The build was interrupted to prioritize another pull request.

@bors-servo
Copy link
Contributor

bors-servo commented Aug 19, 2016

Testing commit 4ede9f6 with merge bf8b7ba...

bors-servo added a commit that referenced this pull request Aug 19, 2016
Remove unused imports.

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

bors-servo commented Aug 19, 2016

@bors-servo bors-servo merged commit 4ede9f6 into master Aug 19, 2016
3 of 4 checks passed
3 of 4 checks passed
dependency-ci Failed dependency checks
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the imports branch Sep 5, 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

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