aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy
Commit message (Collapse)AuthorAgeFilesLines
* Auto merge of #16893 - MortimerGoro:webgl_extensions, r=emiliobors-servo2017-05-191-0/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement WebGL extensions <!-- Please describe your changes on the following line: --> This PR provides the base for implementing WebGL extensions. It comes with the following ones already implemented and passing all related WebGL conformance tests: - OES_texture_float - OES_texture_float_linear - OES_texture_half_float - OES_texture_half_float_linear - OES_vertex_array_object I'll submit other extensions like compressed textures in a separate PR to ease the review process. I included the 5 extensions in this PR because it's easier to show/review how the WebGL extension base works. To pass all OES_texture_float_xxx tests I had to add some missing format conversions in WebGLRenderingContext.rs --- <!-- 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 - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- 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/16893) <!-- Reviewable:end -->
| * Implement WebGL extensions.Imanol Fernandez2017-05-181-0/+1
| |
* | Make some attr values case-insensitive in selectorsSimon Sapin2017-05-181-0/+10
|/ | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1363531
* Implemented Houdini worklets.Alan Jeffrey2017-05-171-0/+1
|
* Make tidy check that opening and closing braces that begin a line do so with ↵coalman2017-04-181-3/+12
| | | | proper alignment.
* Check for merge commits with tidySadman Kazi2017-04-131-4/+17
| | | | | | Increase merge commit check range under travis Add comment on why travis workaround is necessary
* Tidy: Prefer monkey patching sys.path over addsitedir (and some cleanup)Ravi Shankar2017-03-201-19/+20
|
* added tidy check for -> ()lucantrop2017-03-131-0/+2
|
* Rewrite the ban-type lint in PythonJefry Lagrange2017-02-251-0/+6
| | | | | | Delete old rust ban lint and move tests to python tidy Fix ban lint regex and fix test
* Add 'use statements with extraneous spaces' tidy checkvwvww2017-02-231-0/+4
| | | | | | | | | | | | | | | | | | | | | Add 'use statements with extraneous spaces' tidy check I added simple check routine for 'use statements with extraneous spaces' and codes that breaks the check routine in rust_tidy.rs. * Added a code that using 'use statements with extraneous spaces' code in rust_tidy.rs * Added assertion code in test_tidy.py. * check_rust function in tidy.py now recognizes the simple case in the 'use statements with extraneous spaces'. * Ran tidy check on rust code and modified a code(tests/unit/style/parsing/inherited_text.rs) that is not passing on this new tidy check. TODO: this code has to be refactored to support more general cases. - [X] ./mach test-tidy does not report any errors - [X] These changes fix #14898 (github issue number if applicable).
* tidy: Ignore [replace]’d packages in the duplicate package lint.Simon Sapin2017-02-141-0/+2
|
* Tidy checks for styloManish Goregaokar2017-02-081-15/+26
|
* Tidy: Report invalid listed ignored directories and filesUK9922017-01-181-5/+34
|
* Moved tidy's BLOCKED_PACKAGES to the config fileMatthias Devlamynck2017-01-141-23/+6
|
* 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
|
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-27 09:57:40 +0000