aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy/tidy.py
Commit message (Collapse)AuthorAgeFilesLines
...
* 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.
* Allow MIT/Apache-2.0 license as well as MPL-2.0.Alan Jeffrey2016-07-141-5/+6
|
* All our Cargo.toml files should contain an MPL-2.0 license field.Alan Jeffrey2016-07-141-0/+4
|
* [tidy] check for duplicate keys in .json filesOwen Coutts2016-07-081-1/+13
|
* tidy command now ignores files in subdirectories of ignored directoriesNikhil Shagrithaya2016-07-071-1/+1
|
* stylo: Move all binding-generator code to a python script.Emilio Cobos Álvarez2016-07-051-2/+0
| | | | | | | | | | | | | | | | | | | | This not only makes us more consistent with the rest of the codebase but also: * Makes us repeat less code like common flags and all that stuff. * Reduces the noise of the build: You only get the output of the commands on failure or when you pass the -v flag. * Makes you able to select a single kind of build or multiple in the same place. I've basically kept the regen.sh script because of the LIBCLANG_PATH checks, but at least from Linux I don't need them anymore. Also, that logic could be moved to the new script. The whole point of this isn't only making it prettier and easier to use, but also allowing me to write more complex logic in the binding generator scripts, that I will probably need to integrate the DOM enum types we need for animations and such easily (can't be just an include, because that pulls in another header with the same name bringing a lot of DOM and IDL churn).
* Add style check, test, and code fixes for an else brace check.Travis Dean2016-07-041-0/+1
|
* Update web-platform-tests to revision 346d5b51a122f7bb1c7747064499ef281a0200f7Ms2ger2016-06-241-1/+1
|
* Removed unused function and minor cleanupRavi Shankar2016-06-161-76/+63
|
* Auto merge of #11621 - h4xr:mach_fix, r=Wafflespeanutbors-servo2016-06-141-2/+3
|\ | | | | | | | | | | | | | | | | | | | | | | | | Make mach test-tidy consider ignored dirs Made changes so that mach test-tidy considers the ignored directories Fixes #11386 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11621) <!-- Reviewable:end -->
| * Make mach consider ignored dirsSaurabh Badhwar2016-06-151-2/+3
| | | | | | | | Address indentation changes
* | servo: Add an `Info.plist` on the Mac and opt into integrated graphics.Patrick Walton2016-06-081-0/+3
| | | | | | | | | | | | | | | | | | | | | | Discrete GPUs cause power use problems and tend to perform badly with WebRender. See: * https://developer.apple.com/library/mac/qa/qa1734/_index.html * https://reverse.put.as/2013/05/28/gimmedebugah-how-to-embedded-a-info-plist-into-arbitrary-binaries/
* | Auto merge of #11190 - mrmiywj:Root-tidy-checker, r=jdmbors-servo2016-06-051-0/+2
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add &Root<T> checker 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 #11137 (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="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11190) <!-- Reviewable:end -->
| * add &Root<T> checkermrmiywj2016-06-051-0/+2
| | | | | | | | | | | | update rust_tidy tests update rust_tidy.rs and test_tidy.py
* | Generate debug and release Gecko bindings.Cameron McCormack2016-06-031-1/+2
| |
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-02 02:44:44 +0000