Simon Martin
cec661fa86
Issue #9042 : Report incorrect number of spaces around => in the style checker.
2015-12-23 20:54:26 +01:00
Corey Farwell
75d64d3b3f
Print tidy errors as they happen instead of all-at-once
2015-12-14 18:07:11 -05:00
Ms2ger
07d837bd03
Update bincode.
2015-12-08 14:49:38 -05:00
bors-servo
cad77d5f1c
Auto merge of #8824 - Ms2ger:tidy-lock, r=Wafflespeanut
...
Rewrite the Cargo.lock tidy check.
Fixes #8691 .
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8824 )
<!-- Reviewable:end -->
2015-12-08 10:21:55 +05:30
Guillaume Gomez
db3457e498
Improve checking algorithms
2015-12-06 03:34:59 +01:00
Ms2ger
b72c8e5e18
Rewrite the Cargo.lock tidy check.
2015-12-04 17:55:48 +01:00
bors-servo
8b95d7b8d8
Auto merge of #8757 - servo:skia, r=mbrubeck
...
Use skia and deps from crates.io.
This makes the initial download for skia go from a 300 MB git repository to a 5 MB tarball. This should help with issues like #6132 and #7687 .
Fix https://github.com/servo/skia/issues/70
This builds, but the at the moment causes a number of tidy errors for duplicated crates.
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8757 )
<!-- Reviewable:end -->
2015-12-03 12:58:29 +05:30
Simon Sapin
b8a3a64691
Tidy ports/cef/Cargo.lock
...
Despite the rest of ports/cef/ being ignored.
2015-12-02 19:11:08 +01:00
Simon Sapin
aa1eba4f5a
Use skia and deps from crates.io.
...
Fix https://github.com/servo/skia/issues/70
2015-12-02 19:07:59 +01:00
jmr0
e301ff46b2
adding wpt lint script to tidy checks
2015-11-30 10:08:10 -05:00
bors-servo
dbff1ab336
Auto merge of #8692 - GuillaumeGomez:patch-1, r=Wafflespeanut
...
Ensure crate are alphabetically sorted
cc @nox
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8692 )
<!-- Reviewable:end -->
2015-11-28 19:34:11 +05:30
Guillaume Gomez
6e7de62b38
Add check up on extern crate order and sort extern crates alphabetically
2015-11-28 03:11:08 +01:00
Manish Goregaokar
dc0e467945
Upgrade to rustc 1.6.0-nightly (d5fde83ae 2015-11-12)
...
… and libc 0.2 and many other dependencies
2015-11-27 00:15:29 -05:00
bors-servo
bc618b0d53
Auto merge of #8689 - jdm:deps, r=frewsxcv
...
Remove remaining version-conflict exceptions in tidy.py.
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8689 )
<!-- Reviewable:end -->
2015-11-27 09:17:54 +05:30
Josh Matthews
c36cbf58e8
Remove remaining version-conflict exceptions in tidy.py.
2015-11-26 11:26:28 -05:00
jmr0
df49cf2b13
adding tidy rule to warn against use of &String and refactoring instances of &String in codebase
2015-11-25 21:49:12 -05:00
Ms2ger
026994e4be
Ignore sync directories in test-tidy.
2015-11-10 10:29:26 +01:00
Corey Farwell
d1824aea48
Expand tidy to lint parts of /tests/wpt
2015-11-03 17:11:44 -05:00
Anthony Ramine
bd363b009d
Make test-tidy ignore braces surrounded by backticks
2015-10-21 10:59:36 +02:00
Till Schneidereit
499a847141
Whitelist drafts.fxtf.org in tidy.py's webidl spec link check
2015-10-17 22:25:21 +02:00
Gilles Leblanc
d99149701f
Adds a tidy check for single-page HTML specification links
...
Flags links to the single-page WHATWG specification and suggests the URL
for the multi page one.
Fixes #7998
2015-10-15 21:03:58 -04:00
bors-servo
0f597d3890
Auto merge of #7933 - mbrubeck:piston-image, r=glennw
...
Replace libpng and stb_image with PistonDevelopers/image
Fixes #3368 . r? @glennw
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7933 )
<!-- Reviewable:end -->
2015-10-12 15:06:31 -06:00
Corey Farwell
85f2b6fc5b
Replace usage of old-style WHATWG spec links
2015-10-10 11:55:09 -04:00
Matt Brubeck
e2e142aae8
Replace libpng with image everywhere.
2015-10-10 07:43:11 -07:00
Matt Brubeck
b649246fe2
Replace stb_image with PistonDevelopers/image
...
Fixes #3368
2015-10-09 12:23:33 -07:00
Pierre Chevalier
d2481c37d5
Refactor away unnecessary use of regex
...
We don't need regex matching since ": &Vec<" doesn't
contain any special character.
New code reads better.
Fixes #7914 .
2015-10-09 16:43:26 +01:00
Jaydeep
2a99915188
Check for Extra pointer dereferencing. Issue #7640 .
2015-09-26 19:33:29 -07:00
Mathieu Rheaume
705d8f7a1c
Raise max length error over 120 chars not at 120 chars. python/tidy.py
2015-09-21 22:25:32 -04:00
Brandon Fairchild
e924393be8
tidy.py: Check for import blocks separated by whitespace
...
This enables flagging multiple import blocks separated by whitespace
as errors.
Fixes #7381 .
2015-09-19 12:46:01 -04:00
Hugo Thiessard
e81e91207c
fixes #7390 : tidy now check the order of mod declarations even whith attribute
2015-09-18 21:32:00 +02:00
bors-servo
3a97455a65
Auto merge of #7570 - jxs:ignore_length_tidy_script, r=Ms2ger
...
ignore length on .lock files on tidy.py,
closes #7569
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7570 )
<!-- Reviewable:end -->
2015-09-11 03:07:34 -06:00
João Oliveira
b9ffb80b7c
ignore length on .lock files on tidy.py,
...
closes #7569
2015-09-09 14:02:43 +01:00
Corey Farwell
244af42d89
Use OS-agnostic filesystem paths in Python
...
This will eventually need to be done for #1908
2015-09-08 21:16:02 +02:00
bors-servo
3424e234c8
Auto merge of #7499 - nerith:style, r=metajack
...
Improve style nit check for space after a comma
Fixes #7345 .
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7499 )
<!-- Reviewable:end -->
2015-09-02 12:05:58 -06:00
Corey Farwell
31e57cb349
Remove references to no-longer-in-tree Python packages
2015-09-02 09:34:02 -04:00
Corey Farwell
e38df50bbf
Revert "Revert "Auto merge of #7103 - frewsxcv:python-venv, r=metajack" for breaking web-platform-tests."
...
This reverts commit 47d6d958f5
.
2015-09-02 09:34:01 -04:00
bors-servo
8b5418f58f
Auto merge of #7438 - Wafflespeanut:cargo-tidy, r=jdm
...
Tidy can now check for version conflicts...
fixes #7133
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7438 )
<!-- Reviewable:end -->
2015-09-01 14:38:48 -06:00
Brandon Fairchild
24bc6dfb52
python/tidy.py: Ignore macros in style nit check
...
Lines with macros were flagged as errors when checking
for space after a comma.
2015-09-01 16:25:44 -04:00
Brandon Fairchild
0bddd25805
Improve style nit check for space after a comma
...
Only a small number of things that followed a comma were flagged.
This improves the thoroughness of that check.
Fixes #7345 .
2015-09-01 15:02:41 -04:00
Ravi Shankar
da1c8581d9
Tidy could now check for version conflicts!
2015-09-02 00:23:25 +05:30
bors-servo
15de679f11
Auto merge of #7468 - JoshTheGoldfish:Issue7460, r=jdm
...
Making test-tidy check that = have space after them
For issue #7460 . Need to ensure compatibility with #7390 .
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7468 )
<!-- Reviewable:end -->
2015-09-01 08:33:02 -06:00
erneyja
17663315dd
make test-tidy check that = have space after them
2015-09-01 07:18:19 -04:00
Corey Farwell
3a1d140ab5
Enforce linking to spec for method implementations via macros
2015-08-31 21:02:23 -04:00
bors-servo
7474b29510
Auto merge of #7478 - frewsxcv:spec-link-tidy-fix, r=jdm
...
Update spec link pattern to reflect `impl` signature changes
Relevant to #7416
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png " height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7478 )
<!-- Reviewable:end -->
2015-08-31 15:48:24 -06:00
Corey Farwell
20c0af8b7d
Update spec link pattern to reflect impl
signature changes
...
Relevant to #7416
2015-08-31 17:38:23 -04:00
wilmoz
64ac4f175f
Make test-tidy check that braces have spaces before or after them
2015-08-31 12:30:55 -05:00
Ravi Shankar
6ef5c8eec2
tidy will now show its expectation of uses
2015-08-28 13:14:45 +05:30
wilmoz
63851f2650
Forbid multiline imports
2015-08-25 12:27:42 -05:00
Corey Farwell
662b64ffe4
Make tidy search for files recursively again
...
In #7348 `os.walk` was replaced with `os.listdir`. The latter is not
recursive, which results in only the root directory files getting linted
The changes to `ignored_files` are needed because calling `os.walk(".")`
results in `./` getting prefixed before each path
2015-08-24 20:34:01 -04:00
João Oliveira
69ddedc568
update tidy.py remove collect_file_names:
...
call python os functions inline
2015-08-25 00:05:37 +01:00