aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy_tests
Commit message (Collapse)AuthorAgeFilesLines
...
| * tidy: checking directories for unexpected file extensionsGregory2016-10-026-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | using word license in test name for consistencyMatt Schmoyer2016-10-011-1/+1
|/
* Issue #13350 Report tidy error for space after (Hugo Thiessard2016-09-232-0/+11
|
* Fix ordering `use` statements with bracesUK9922016-09-091-0/+1
|
* Tidy: Handle empty filesUK9922016-08-292-0/+5
|
* Check for license only in Cargo.tomlUK9922016-08-252-1/+1
|
* Some cleanups in tidyRavi Shankar2016-08-231-3/+3
|
* Check for errors in tidy config fileUK9922016-08-202-0/+20
|
* Check for blank line after shebang and grab actual comment block instead of ↵UK9922016-08-122-0/+9
| | | | hard-coded one
* Improve tidy's license validation logicedunham2016-08-122-0/+9
| | | | | | | | | | | 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.
* Add lint for `[` instead of `[[` in shells scriptsAneesh Agrawal2016-08-052-0/+6
|
* Add lint to ensure substitutions use the full formAneesh Agrawal2016-08-052-0/+2
| | | | | 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-052-0/+3
| | | | | | 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.
* Auto merge of #12295 - jimberlage:12158/shell-linting, r=aneeshusabors-servo2016-08-042-0/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add linting for shell scripts <!-- Please describe your changes on the following line: --> 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. --- <!-- 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 #12158 (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/12295) <!-- Reviewable:end -->
| * Add linting for shell scriptsJim Berlage2016-07-222-0/+13
| | | | | | | | | | | | 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 #12567 - servo:tidy-self-test-result, r=jdmbors-servo2016-07-241-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | Return failure if the tidy self-tests fail. <!-- 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/12567) <!-- Reviewable:end -->
| * | Return failure if the tidy self-tests fail.Ms2ger2016-07-231-1/+1
| |/
* / Issue #12564: Fix "./mach test-tidy --self-test".Simon Martin2016-07-231-3/+4
|/
* [tidy] check for duplicate keys in .json filesOwen Coutts2016-07-082-0/+12
|
* Auto merge of #12317 - tallowen:add-tidy-test, r=jdmbors-servo2016-07-072-0/+9
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [tidy] Test json parsing functionality We didn't have a test for json parsing so lets start with that --- <!-- 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 <!-- 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/12317) <!-- Reviewable:end -->
| * [tidy] Add test for existing json functionalityOwen Coutts2016-07-072-0/+9
| |
* | Added a subdirectory with a file to test whether tidy command ignores files ↵Nikhil Shagrithaya2016-07-071-0/+0
|/ | | | in subdirectories
* Add style check, test, and code fixes for an else brace check.Travis Dean2016-07-042-0/+7
|
* Tidy test for ignoring attributesRavi Shankar2016-06-161-0/+1
|
* Removed unused function and minor cleanupRavi Shankar2016-06-161-1/+1
|
* Auto merge of #11621 - h4xr:mach_fix, r=Wafflespeanutbors-servo2016-06-142-0/+10
|\ | | | | | | | | | | | | | | | | | | | | | | | | 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-152-0/+10
| | | | | | | | Address indentation changes
* | Auto merge of #11190 - mrmiywj:Root-tidy-checker, r=jdmbors-servo2016-06-052-1/+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-052-1/+2
| | | | | | | | | | | | update rust_tidy tests update rust_tidy.rs and test_tidy.py
* | Warn about empty lines following an open brace.Josh Matthews2016-05-271-0/+1
| |
* | Report use statements that use {} with only one entryCullen Rhodes2016-05-272-0/+2
|/
* add tidy test for 'missing space after :' in stuctsaeischeid2016-05-112-0/+6
|
* Adding parameter to tidy for skipping print text when running testsaskeing2016-04-261-10/+10
|
* Test for tidy.check_lock function #9152Maciej Skrzypkowski2016-04-252-0/+33
|
* Auto merge of #10786 - zwn:tidy-modelines, r=Manishearthbors-servo2016-04-222-0/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | tidy check for vim and emacs modelines See #10719. It skips *.webidl files for now since I am not sure where they come from and if they should be edited in tree or not. <!-- 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/10786) <!-- Reviewable:end -->
| * tidy check for vim and emacs modelines.Zbynek Winkler2016-04-212-0/+13
| | | | | | | | See https://github.com/servo/servo/issues/10719.
* | Report lines starting with &&.Zbynek Winkler2016-04-222-1/+5
| |
* | Ensure that we aren't missing any errors in the tidy self-test.Josh Matthews2016-04-201-0/+13
|/
* Modified test_spec_link of test_tidy for packaged tidyaskeing2016-04-172-81/+4
|
* Test for check_spec function, for tidy.py #9152Maciej Skrzypkowski2016-04-171-0/+5
|
* Modified the servo_tidy_tests for both "./mach" and "make test"askeing2016-04-141-1/+3
| | | | - fix `cd python/tidy; make test` fail issue
* Package tidyaskeing2016-04-1411-0/+239
- Modified the testing commands - Added the requirements - Moved python/tidy/tests to python/tidy/servo_tidy_tests for self tidy tests
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-30 11:59:08 +0000