aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy_tests
Commit message (Collapse)AuthorAgeFilesLines
* Convert tidy to a non-egg Python packageMartin Robinson2023-06-1542-722/+0
| | | | | | | | | | | It seems that servo-tidy is only used by webrender in my GitHub searches. WebRender could simply use `rustfmt` and the tidy on pypi hasn't been updated since 2018. Converting tidy to a normal Python package removes the maintenance burden of continually fixing the easy install configuration. Fixes #29094. Fixes #29334.
* Merge forbidden panic check into test-tidyMartin Robinson2023-03-223-1/+7
| | | | | | | | | This cleans up the GitHub actions yaml a bit and ensures that developers are running this check locally before submitting changes. In addition, it allows adding tests for this check. Finally, this change fixes the tidy tests by upgrading voluptuous for Python 3.10 as well as by reverting an inadvertent change for NixOS compatibility on one of the dummy testing files.
* Use `#!/usr/bin/env bash` in shell scripts for NixOS compatibilityyvt2021-06-141-1/+1
| | | | Signed-off-by: yvt <i@yvt.jp>
* Fix output of test-tidy with duplicated packages.Josh Matthews2020-05-202-5/+9
|
* Remove the lint added in https://github.com/servo/servo/pull/14589Simon Sapin2020-04-082-24/+0
| | | | | Per https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=a8a5a716b3bdf52fa7960a1e4ee89add it looks like the drop order of temporaries was changed at some point and this pattern is no longer problematic.
* Remove azure canvas backendDaniel Alley2020-02-251-4/+4
| | | | closes #25833
* Add lint check for `&DomRoot<T>`lberrymage2019-12-212-1/+2
| | | | | `&DomRoot<T> is strictly less expressive than `&T`, so using it is pointless.
* Make `mach test-tidy --self-test` compatible with Python3marmeladema2019-12-141-69/+69
|
* report when blocked package exception is not neededSiddhartha Mishra2019-09-222-0/+45
|
* Remove regex-based checks in servo-tidy that are covered by rustfmtSimon Sapin2019-06-221-25/+0
|
* Treat url strings are unsplittableShanavas M2019-01-081-0/+1
| | | | Fixes #22498
* Update MPL license to https (part 1)Jan Andre Ikenmeyer2018-11-1911-11/+11
|
* Remove useless `use crate_name;` imports.Simon Sapin2018-11-081-1/+0
| | | | A `crate_name::foo` path always works in 2018
* Fix test-tidy self testPyfisch2018-11-071-6/+0
| | | | | Install rustup on first Travis job. Only use rustfmt::skip as an outer attribute.
* Add test that actually tests the testTimur Borkhodoev2018-02-022-105/+2
|
* Add correct file test for manifest testTimur Borkhodoev2018-02-022-2/+114
|
* Add self-test and modify tidy to accomodate the testTimur Borkhodoev2018-02-011-0/+5
|
* Update buildbot_steps lint to handle env variablesAneesh Agrawal2018-01-081-2/+2
| | | | | | | | | | https://github.com/servo/saltfs/pull/687 added support for specifying environment variables in `buildbot_steps.yml`. Update the servo-tidy buildbot_steps.yml linter to reflect this. Use the voluptuous Python library (BSD 3-clause license) for validation in lieu of a much larger hand-written implementation. Update the tidy self tests to take into account the new error messages.
* Report an errror if a package has duplicates allowed but there are no duplicatesMaxim Novikov2017-11-211-0/+18
| | | | Resolves: #19306
* Fix commonmark Markdown warnings in docs, part 1Matt Brubeck2017-10-171-2/+2
| | | | | | | | Fixes warnings from rust-lang/rust#44229 when `--enable-commonmark` is passed to rustdoc. This is mostly a global find-and-replace for bare URIs on lines by themselves in doc comments.
* Rename DOMRefCell<T> to DomRefCell<T>Anthony Ramine2017-09-262-4/+4
| | | | | | | | I don't want to do such a gratuitous rename, but with all the other types now having "Dom" as part of their name, and especially with "DomOnceCell", I feel like the other cell type that we already have should also follow the convention. That argument loses weight though when we realise there is still DOMString and other things.
* Rename MutJS<T> to MutDom<T>Anthony Ramine2017-09-263-4/+4
|
* Rename JS<T> to Dom<T>Anthony Ramine2017-09-262-4/+4
|
* Fix some messages in the tidy scriptAnthony Ramine2017-09-262-2/+2
| | | | It's MutJS<T>, not MutJS<JS<T>>.
* Make tidy aware of Rust multiline stringsMichael Droettboom2017-09-212-0/+33
| | | | | | As a result of tighter and more correct handling of character literals, this now catches a few kinds of syntax involving lifetimes that were previously missed, so those have been updated.
* Automatically verify that derive() lists are alphabetically orderedClément DAVID2017-08-232-0/+2
|
* tidy: add extra space after keyword checkPavel Sergeev2017-07-202-0/+8
|
* Upgrade to the latest version of WebRenderMartin Robinson2017-07-131-1/+1
|
* Bump euclid to 0.14.Nicolas Silva2017-06-141-1/+1
|
* Count number of braces on a line in .Alan Jeffrey2017-06-072-0/+24
|
* Make tidy check that opening and closing braces that begin a line do so with ↵coalman2017-04-182-0/+11
| | | | proper alignment.
* Allow using servo as dependency, by fixing Cargo.toml of servo-test-tidy.강동윤2017-03-271-0/+3
|
* added tidy check for -> ()lucantrop2017-03-132-1/+2
|
* Fix Cargo.toml in servo_tidy_testsUK9922017-02-271-1/+4
|
* Rewrite the ban-type lint in PythonJefry Lagrange2017-02-253-0/+48
| | | | | | 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-232-0/+3
| | | | | | | | | | | | | | | | | | | | | 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: Report invalid listed ignored directories and filesUK9922017-01-182-2/+8
|
* Moved tidy's BLOCKED_PACKAGES to the config fileMatthias Devlamynck2017-01-141-0/+2
|
* Rust tidy: disallow assignment = in the beginning of lineJure Podgoršek2017-01-062-0/+4
|
* Tidy: Check Cargo.lock for packages with same version and different sourcesUK9922016-12-262-7/+37
|
* Add a tidy check for problematic match cases in script_thread.rsJosh Matthews2016-12-162-0/+24
|
* Auto merge of #14051 - birryree:tidy-check-buildbot-steps, r=aneeshusabors-servo2016-12-084-0/+31
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-084-0/+31
| | | | | | | | | | | | | | | | | | 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
* | Add tests for the new LintRunnerRavi Shankar2016-11-117-0/+46
| |
* | Move file list iterator to a new moduleRavi Shankar2016-11-101-4/+3
|/
* Check for JSON key non-duplication and orderYuki Izumi2016-11-072-0/+13
|
* Auto merge of #13986 - iamrohit7:tidy-feature-attrs, r=Wafflespeanutbors-servo2016-11-012-0/+22
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 -->
| * Tests alphabetical ordering of #![feature(...)]Rohit Burra2016-11-012-0/+22
| |
* | test-tidy: Check for space between function name and `(`Daan Sprenkels2016-10-312-1/+4
|/ | | | | | | | 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.
* Auto merge of #13447 - gterzian:check_webidls_folder_files, r=Wafflespeanutbors-servo2016-10-026-0/+12
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 -->
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-30 07:04:18 +0000