aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy/tidy.py
Commit message (Collapse)AuthorAgeFilesLines
...
* Rust tidy: disallow assignment = in the beginning of lineJure Podgoršek2017-01-061-0/+2
|
* Added test-tidy check for blocked packages.Alan Jeffrey2017-01-051-0/+35
|
* Fix tidy json alphabetical order checkUK9922016-12-291-1/+1
|
* Tidy: Check Cargo.lock for packages with same version and different sourcesUK9922016-12-261-42/+23
|
* Tidy: Simplify path normalizationUK9922016-12-261-15/+11
|
* Add a tidy check for problematic match cases in script_thread.rsJosh Matthews2016-12-161-0/+3
|
* Auto merge of #14051 - birryree:tidy-check-buildbot-steps, r=aneeshusabors-servo2016-12-081-4/+58
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding linting checks for buildbot_steps.yml This pull request adds some tidy checks around YAML files, and specifically `buildbot_steps.yml`. Tidy checks added: * YAML files are checked for well-formedness/parse-ability * Whether a YAML file has duplicate keys * Whether a `buildbot_steps.yml` file contains only mappings to list-of-strings. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [x] These changes fix #13838 (github issue number if applicable). <!-- Either: --> - [X] There are tests for these changes OR <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> …ing checking for correct mappings and duplicate YAML keys. Added unit tests to test_tidy.py. <!-- 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/14051) <!-- Reviewable:end -->
| * Add tidy linting checks for buildbot_steps.ymlWilliam Lee2016-12-081-4/+58
| | | | | | | | | | | | | | | | | | This commit adds tidy checks for buildbot_steps.yml, as well as unit tests. These checks include: * Checking buildbot_steps.yml can be parsed by a YAML loader * buildbot_steps.yml does not contain duplicate keys * buildbot_steps.yml keys map to a list of strings
* | Implement tidy commit message testSijmen Schoon2016-12-031-1/+19
| | | | | | Tests for work in progress commits.
* | Move to Cargo workspacesAnthony Ramine2016-11-281-0/+2
| |
* | Support basic immutable CSSOMManish Goregaokar2016-11-151-1/+1
| |
* | Isolate the WPT lint and make use of the LintRunnerRavi Shankar2016-11-111-27/+4
| |
* | Add a generic LintRunner object for running custom lintsRavi Shankar2016-11-111-0/+49
| |
* | Move file list iterator to a new moduleRavi Shankar2016-11-101-38/+50
|/
* Check for JSON key non-duplication and orderYuki Izumi2016-11-071-3/+18
|
* Auto merge of #13986 - iamrohit7:tidy-feature-attrs, r=Wafflespeanutbors-servo2016-11-011-0/+25
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make test-tidy check for alphabetical ordering of #![feature(...)] statements <!-- Please describe your changes on the following line: --> <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [x] These changes fix #13954 (github issue number if applicable). <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/13986) <!-- Reviewable:end -->
| * Alphabetical order for #![feature(...)] in lib.rsRohit Burra2016-11-011-0/+25
| |
* | Auto merge of #13988 - dsprenkels:tidy-fncalls, r=Wafflespeanutbors-servo2016-10-311-0/+7
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test-tidy: Check for space between function name and `(` <!-- Please describe your changes on the following line: --> Disallow an extraneous space in a function call between the function name and the opening parenthesis in Rust code, while ignoring macro declarations. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy --all` does not report any errors - [x] `./mach test-tidy --self` does not report any errors - [x] These changes fix #13980 <!-- Either: --> - [x] There are tests for these changes, these are written in - `python/tidy/servo_tidy_tests/rust_tidy.rs` - `python/tidy/servo_tidy_tests/test_tidy.py` - [ ] These changes do not require tests <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/13988) <!-- Reviewable:end -->
| * | test-tidy: Check for space between function name and `(`Daan Sprenkels2016-10-311-0/+7
| |/ | | | | | | | | | | | | | | Disallow an extraneous space in a function call between the function name and the opening parenthesis in Rust code, while ignoring macro declarations. This commit fixes #13980.
* / Reinstate the wpt lint checking.Ms2ger2016-10-311-6/+10
|/
* Update to use the new Android library build rules.Mátyás Mustoha2016-10-281-1/+1
|
* Don't check untracked file in tidyShing Lyu2016-10-281-4/+1
|
* add SVG2 draft link to WEBIDL_STANDARDS test-tidy listAlexandrov Sergey2016-10-071-0/+1
|
* Auto merge of #13447 - gterzian:check_webidls_folder_files, r=Wafflespeanutbors-servo2016-10-021-3/+30
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Have tidy ensure that the there are no extra files in the webidls folder #13427 <!-- Please describe your changes on the following line: --> Have tidy ensure that the there are no extra files in the webidls folder #13427 --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [x] These changes fix #13427 (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/13447) <!-- Reviewable:end -->
| * tidy: checking directories for unexpected file extensionsGregory2016-10-021-3/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | update config, check_dir func to use config pass only_changed_files and exclude_dirs as args simplify config better conditional remove unused default, amend error message undo doc change
* | Avoid unecessary loop evaluation on empty lines for tidy check_licenseMatt Schmoyer2016-10-011-2/+3
|/
* Replace instances of old ServiceWorker specification URL with new oneVignesh Sarma K (വിഘ്നേഷ് ശ൪മ കെ)2016-09-251-1/+0
| | | | | | The old specification URL is https://slightlyoff.github.io/ServiceWorker/spec/service_worker/ has been replaced by the new one at https://w3c.github.io/ServiceWorker/.
* Issue #13350 Report tidy error for space after (Hugo Thiessard2016-09-231-0/+1
|
* Fix ordering `use` statements with bracesUK9922016-09-091-4/+7
|
* Tidy: Handle empty filesUK9922016-08-291-0/+3
|
* Check for license only in Cargo.tomlUK9922016-08-251-4/+3
|
* Some cleanups in tidyRavi Shankar2016-08-231-40/+31
|
* Check for errors in tidy config fileUK9922016-08-201-1/+49
|
* Tidy config fileUK9922016-08-201-55/+56
|
* Ignore TOML failing to parse Cargo.lock, due to a bugVladimir Vukicevic2016-08-171-2/+12
|
* Don't enforce line length in Cargo.toml filesVladimir Vukicevic2016-08-171-2/+3
|
* Auto merge of #12781 - UK992:tidycheck-rebased, r=Wafflespeanutbors-servo2016-08-141-12/+43
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | Improve tidy's license validation logic Rebased and fixed https://github.com/servo/servo/pull/10721, which is inactive for months. Fixes https://github.com/servo/servo/issues/10716 r? @larsbergstrom or @edunham <!-- 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/12781) <!-- Reviewable:end -->
| * Add support colored text in Windows ConsoleUK9922016-08-131-0/+3
| |
| * Check for blank line after shebang and grab actual comment block instead of ↵UK9922016-08-121-3/+13
| | | | | | | | hard-coded one
| * Improve tidy's license validation logicedunham2016-08-121-12/+30
| | | | | | | | | | | | | | | | | | | | | | fixes https://github.com/servo/servo/issues/10716 I took the lazy way out and hardcoded the size of block we examine for licenses. fixes https://github.com/servo/servo/issues/10719 Includes tests for new functionality.
* | Update ordered-float.Ms2ger2016-08-111-1/+1
| |
* | Issue #12421: tidy should also check .html filesSimon Martin2016-08-091-4/+9
| |
* | Prefer length and percentage for word spacingRavi Shankar2016-08-091-1/+1
|/
* Add lint for `[` instead of `[[` in shells scriptsAneesh Agrawal2016-08-051-0/+3
|
* Add lint to ensure substitutions use the full formAneesh Agrawal2016-08-051-0/+7
| | | | | Check that any variable substitutions use the full ${VAR} form, not just $VAR (but don't check for quoting yet).
* Add lint for backticks in shell scriptsAneesh Agrawal2016-08-051-10/+17
| | | | | | The "$(some_command arg1 arg2)" form is preferred to the `some_command arg1 arg2` form because it nests unambiguously. Add a lint for this to tidy.
* Add linting for shell scriptsJim Berlage2016-07-221-3/+39
| | | | | | This changes tidy to check shell scripts for the proper shebang and options. It does not check that variables are formatted correctly. It also adds a check for the MPL 2.0 license in shell scripts.
* Auto merge of #12511 - asajeffrey:test-tidy-dep-licenses, r=larsbergstrombors-servo2016-07-211-2/+26
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added test-tidy for dependent licenses. <!-- Please describe your changes on the following line: --> Add a test-tidy lint for dependency licenses. Note that #12507 should land first. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes do not require tests because we don't test our lints <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- 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/12511) <!-- Reviewable:end -->
| * Added test-tidy for dependent licenses.Alan Jeffrey2016-07-211-2/+26
| |
* | Add the append method for the Headers API for the Fetch APIJeena Lee2016-07-191-0/+1
|/ | | | This commit will add the append method and associated helper functions, and introduce any necessary changes for it.
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-02 08:52:50 +0000