aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy.py
Commit message (Collapse)AuthorAgeFilesLines
...
* | 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
| |/
* | Enforce linking to spec for method implementations via macrosCorey Farwell2015-08-311-3/+20
| |
* | Auto merge of #7478 - frewsxcv:spec-link-tidy-fix, r=jdmbors-servo2015-08-311-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 -->
| * | Update spec link pattern to reflect `impl` signature changesCorey Farwell2015-08-311-1/+1
| |/ | | | | | | Relevant to #7416
* / Make test-tidy check that braces have spaces before or after themwilmoz2015-08-311-0/+10
|/
* tidy will now show its expectation of usesRavi Shankar2015-08-281-1/+4
|
* Forbid multiline importswilmoz2015-08-251-2/+1
|
* Make tidy search for files recursively againCorey Farwell2015-08-241-16/+16
| | | | | | | | 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
* update tidy.py remove collect_file_names:João Oliveira2015-08-251-14/+4
| | | | call python os functions inline
* Auto merge of #7118 - g-k:tidy-check-unused-html-files, r=jdmbors-servo2015-08-241-3/+21
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add tidy check for unused reftest html files Refs: https://github.com/servo/servo/issues/7078 Sample output: ``` $ time ./mach test-tidy tests/ref/background_image_a.html not used or commented out in basic.list tests/ref/background_image_ref.html not used or commented out in basic.list tests/ref/canvas_linear_gradient_a.html not used or commented out in basic.list tests/ref/canvas_linear_gradient_ref.html not used or commented out in basic.list tests/ref/canvas_radial_gradient_a.html not used or commented out in basic.list tests/ref/canvas_radial_gradient_ref.html not used or commented out in basic.list tests/ref/inline_border_a.html not used or commented out in basic.list tests/ref/inline_border_b.html not used or commented out in basic.list tests/ref/inline_text_align_a.html not used or commented out in basic.list tests/ref/inline_text_align_b.html not used or commented out in basic.list tests/ref/link_style_dynamic_addition.html not used or commented out in basic.list tests/ref/link_style_dynamic_addition_ref.html not used or commented out in basic.list tests/ref/overflow_position_abs_inside_normal_a.html not used or commented out in basic.list tests/ref/overflow_position_abs_inside_normal_b.html not used or commented out in basic.list tests/ref/overflow_position_abs_simple_a.html not used or commented out in basic.list tests/ref/overflow_position_abs_simple_b.html not used or commented out in basic.list tests/ref/position_fixed_a.html not used or commented out in basic.list tests/ref/position_fixed_b.html not used or commented out in basic.list tests/ref/position_fixed_simple_a.html not used or commented out in basic.list tests/ref/position_fixed_simple_b.html not used or commented out in basic.list tests/ref/position_fixed_static_y_a.html not used or commented out in basic.list tests/ref/position_fixed_static_y_b.html not used or commented out in basic.list tests/ref/style_is_in_doc.html not used or commented out in basic.list tests/ref/style_is_in_doc_ref.html not used or commented out in basic.list tests/ref/table_specified_width_a.html not used or commented out in basic.list tests/ref/table_specified_width_ref.html not used or commented out in basic.list tests/ref/text_decoration_propagation_a.html not used or commented out in basic.list tests/ref/text_decoration_propagation_b.html not used or commented out in basic.list tests/ref/text_shadow_multiple_shadows_a.html not used or commented out in basic.list tests/ref/text_shadow_multiple_shadows_ref.html not used or commented out in basic.list tests/ref/viewport_percentage_vmin_vmax_b.html not used or commented out in basic.list tests/ref/viewport_percentage_vw_vh_b.html not used or commented out in basic.list tests/ref/white_space_intrinsic_sizes_a.html not used or commented out in basic.list tests/ref/white_space_intrinsic_sizes_ref.html not used or commented out in basic.list tests/ref/fonts/takao-p-gothic/COPYING.html not used or commented out in basic.list tests/ref/iframe/multiple_external_child.html not used or commented out in basic.list ... ``` I thought it might be helpful to say which files aren't used in `basic.list`. `./mach test-tidy` is a second or two slower on my laptop. Not sure if this counts as a warning or an error that should return 1. Not sure whether unused file output should go before or after the line specific errors. <!-- Reviewable:start --> [<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7118) <!-- Reviewable:end -->
| * Add tidy check for unused reftest html filesGreg Guthe2015-08-191-3/+21
| | | | | | | | Refs: https://github.com/servo/servo/issues/7078
* | Mention max length in tidy check_length. Fixes #7309OneKorg2015-08-211-2/+3
| |
* | check for uses to be sortedJohann Tuffe2015-08-201-2/+16
|/
* Deal with /* /* when checking syntax.Josh Matthews2015-08-161-8/+7
|
* Tighten up checks for spaces around colons to deal with trait inheritance ↵Josh Matthews2015-08-161-3/+5
| | | | syntax.
* Deal with multiline comments and reduce false positives for common syntax ↵Josh Matthews2015-08-161-10/+41
| | | | overlaps. Improve error messages.
* Deal with multiline strings and tighten up syntax nit error messages.Josh Matthews2015-08-161-19/+25
|
* Check for simple Rust style nits in tidy.pywilmoz2015-08-161-1/+51
|
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-29 10:24:46 +0000