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

Update `servo-tidy` to 0.3.0 #15731

Merged
merged 1 commit into from Feb 25, 2017

Conversation

Projects
None yet
4 participants
@UK992
Copy link
Contributor

commented Feb 24, 2017

r? @wafflespeanut

It's ready to publishing to PyPI.


This change is Reviewable

@highfive

This comment has been minimized.

Copy link

commented Feb 24, 2017

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/tidy/README.rst, python/tidy/HISTORY.rst, python/tidy/setup.py

@UK992 UK992 force-pushed the UK992:tidy branch 3 times, most recently from 481fdcd to af95215 Feb 24, 2017

- Added checking for alphabetical order for JSON keys and ``#![feature(...)]``
- Improved duplicated packages checking
- Improved ordering ``use`` statements
- And many others improvements

This comment has been minimized.

Copy link
@wafflespeanut

wafflespeanut Feb 25, 2017

Member

Nit: other

And, you missed mine - A lint runner for running custom lints 😄

This comment has been minimized.

Copy link
@UK992

UK992 Feb 25, 2017

Author Contributor

done!

@wafflespeanut

This comment has been minimized.

Copy link
Member

commented Feb 25, 2017

r=me after addressing that :)

@bors-servo delegate+

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 2017

✌️ @UK992 can now approve this pull request

@UK992 UK992 force-pushed the UK992:tidy branch from af95215 to 9f55e66 Feb 25, 2017

@wafflespeanut

This comment has been minimized.

Copy link
Member

commented Feb 25, 2017

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 2017

📌 Commit 9f55e66 has been approved by Wafflespeanut

@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 2017

⌛️ Testing commit 9f55e66 with merge 524ba6a...

bors-servo added a commit that referenced this pull request Feb 25, 2017

Auto merge of #15731 - UK992:tidy, r=Wafflespeanut
Update `servo-tidy` to 0.3.0

r? @wafflespeanut

It's ready to publishing to PyPI.

<!-- 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/15731)
<!-- Reviewable:end -->
@bors-servo

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 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-gnu-dev, windows-msvc-dev
Approved by: Wafflespeanut
Pushing 524ba6a to master...

@bors-servo bors-servo merged commit 9f55e66 into servo:master Feb 25, 2017

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.