aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy.py
Commit message (Collapse)AuthorAgeFilesLines
* Auto merge of #9284 - bholley:bootstrap_stylo, r=SimonSapinbors-servo2016-01-191-0/+1
|\ | | | | | | | | | | | | | | | | | | Add Partial Implementation of style DOM traits for Gecko, and some basic glue to let Gecko call into Servo The wrapper stuff is partially-complete, modulo some unimplemented methods. The glue code is just a toy for now. Regardless, I think it's worth getting some of this stuff in-tree to minimize breakage. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9284) <!-- Reviewable:end -->
| * Basic wrappers and glue for GeckoLib.Bobby Holley2016-01-131-0/+1
| |
* | Remove legacy reftest lints.Ms2ger2016-01-141-50/+1
| |
* | Call WPT lint directly in tidy, fixes #9189PythonNut2016-01-121-5/+6
|/
* Fix literal braces in string formatting in tidy.py, fixes #9151PythonNut2016-01-111-4/+4
|
* add a --faster option to ./mach test-tidyDaan Sprenkels2016-01-071-7/+28
| | | | | | which will - only check files changed since the last merge by bors - and skip the wpt-lint
* tidy.py: made `check_rust(file_name, lines)` more readableDaan Sprenkels2015-12-311-89/+67
| | | | | also added a rule that requires a space before '-' (except in cases of scientific notation) fixes #8733
* tidy.py: split checking functions into {normal,line}-checking categoriesDaan Sprenkels2015-12-261-27/+29
| | | | | | | | This is done, because the majority of the functions splits the lines, and it would be better to do this just once. This commit also optimised the licence checking function slightly by only checking the fist lines of the file (where the amount of lines is the line span of the longest license)
* Issue #9042: Report incorrect number of spaces around => in the style checker.Simon Martin2015-12-231-3/+13
|
* Print tidy errors as they happen instead of all-at-onceCorey Farwell2015-12-141-6/+8
|
* Update bincode.Ms2ger2015-12-081-1/+1
|
* Auto merge of #8824 - Ms2ger:tidy-lock, r=Wafflespeanutbors-servo2015-12-081-35/+42
|\ | | | | | | | | | | | | | | | | | | 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 -->
| * Rewrite the Cargo.lock tidy check.Ms2ger2015-12-041-35/+42
| |
* | Improve checking algorithmsGuillaume Gomez2015-12-061-46/+50
|/
* Auto merge of #8757 - servo:skia, r=mbrubeckbors-servo2015-12-031-3/+12
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | 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 -->
| * Tidy ports/cef/Cargo.lockSimon Sapin2015-12-021-0/+2
| | | | | | | | Despite the rest of ports/cef/ being ignored.
| * Use skia and deps from crates.io.Simon Sapin2015-12-021-3/+10
| | | | | | | | Fix https://github.com/servo/skia/issues/70
* | adding wpt lint script to tidy checksjmr02015-11-301-2/+12
| |
* | Auto merge of #8692 - GuillaumeGomez:patch-1, r=Wafflespeanutbors-servo2015-11-281-3/+16
|\ \ | |/ |/| | | | | | | | | | | | | | | 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 -->
| * Add check up on extern crate order and sort extern crates alphabeticallyGuillaume Gomez2015-11-281-3/+16
| |
* | Upgrade to rustc 1.6.0-nightly (d5fde83ae 2015-11-12)Manish Goregaokar2015-11-271-1/+1
| | | | | | | | … and libc 0.2 and many other dependencies
* | Auto merge of #8689 - jdm:deps, r=frewsxcvbors-servo2015-11-271-1/+1
|\ \ | |/ |/| | | | | | | | | | | 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 -->
| * Remove remaining version-conflict exceptions in tidy.py.Josh Matthews2015-11-261-1/+1
| |
* | adding tidy rule to warn against use of &String and refactoring instances of ↵jmr02015-11-251-0/+4
|/ | | | &String in codebase
* Ignore sync directories in test-tidy.Ms2ger2015-11-101-0/+2
|
* Expand tidy to lint parts of /tests/wptCorey Farwell2015-11-031-1/+4
|
* Make test-tidy ignore braces surrounded by backticksAnthony Ramine2015-10-211-4/+4
|
* Whitelist drafts.fxtf.org in tidy.py's webidl spec link checkTill Schneidereit2015-10-171-0/+1
|
* Adds a tidy check for single-page HTML specification linksGilles Leblanc2015-10-151-2/+10
| | | | | | | Flags links to the single-page WHATWG specification and suggests the URL for the multi page one. Fixes #7998
* Auto merge of #7933 - mbrubeck:piston-image, r=glennwbors-servo2015-10-121-1/+3
|\ | | | | | | | | | | | | | | | | | | 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 -->
| * Replace libpng with image everywhere.Matt Brubeck2015-10-101-2/+1
| |
| * Replace stb_image with PistonDevelopers/imageMatt Brubeck2015-10-091-1/+4
| | | | | | | | Fixes #3368
* | Replace usage of old-style WHATWG spec linksCorey Farwell2015-10-101-2/+1
| |
* | Refactor away unnecessary use of regexPierre Chevalier2015-10-091-1/+1
|/ | | | | | | | | We don't need regex matching since ": &Vec<" doesn't contain any special character. New code reads better. Fixes #7914.
* Check for Extra pointer dereferencing. Issue #7640.Jaydeep2015-09-261-0/+5
|
* Raise max length error over 120 chars not at 120 chars. python/tidy.pyMathieu Rheaume2015-09-211-1/+1
|
* tidy.py: Check for import blocks separated by whitespaceBrandon Fairchild2015-09-191-7/+30
| | | | | | | This enables flagging multiple import blocks separated by whitespace as errors. Fixes #7381.
* fixes #7390 : tidy now check the order of mod declarations even whith attributeHugo Thiessard2015-09-181-0/+25
|
* Auto merge of #7570 - jxs:ignore_length_tidy_script, r=Ms2gerbors-servo2015-09-111-2/+4
|\ | | | | | | | | | | | | | | | | | | 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 -->
| * ignore length on .lock files on tidy.py,João Oliveira2015-09-091-2/+4
| | | | | | | | closes #7569
* | Use OS-agnostic filesystem paths in PythonCorey Farwell2015-09-081-13/+13
|/ | | | This will eventually need to be done for #1908
* Auto merge of #7499 - nerith:style, r=metajackbors-servo2015-09-021-2/+2
|\ | | | | | | | | | | | | | | | | | | 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 -->
| * python/tidy.py: Ignore macros in style nit checkBrandon Fairchild2015-09-011-1/+1
| | | | | | | | | | Lines with macros were flagged as errors when checking for space after a comma.
| * Improve style nit check for space after a commaBrandon Fairchild2015-09-011-1/+1
| | | | | | | | | | | | | | Only a small number of things that followed a comma were flagged. This improves the thoroughness of that check. Fixes #7345.
* | Remove references to no-longer-in-tree Python packagesCorey Farwell2015-09-021-4/+0
| |
* | Revert "Revert "Auto merge of #7103 - frewsxcv:python-venv, r=metajack" for ↵Corey Farwell2015-09-021-8/+1
| | | | | | | | | | | | breaking web-platform-tests." This reverts commit 47d6d958f58f5011742a18abcdd5a76bf4390966.
* | Auto merge of #7438 - Wafflespeanut:cargo-tidy, r=jdmbors-servo2015-09-011-3/+43
|\ \ | |/ |/| | | | | | | | | | | | | | | 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 -->
| * Tidy could now check for version conflicts!Ravi Shankar2015-09-021-3/+43
| |
* | Auto merge of #7468 - JoshTheGoldfish:Issue7460, r=jdmbors-servo2015-09-011-18/+34
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 -->
| * | make test-tidy check that = have space after themerneyja2015-09-011-18/+34
| |/
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-14 13:34:35 +0000