aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/test.py
Commit message (Collapse)AuthorAgeFilesLines
* Remove `resources/prefs.json` (#34999)Martin Robinson2025-01-161-16/+0
| | | | | This file is not used any longer. Signed-off-by: Martin Robinson <mrobinson@igalia.com>
* tidy: Make the tidy tests pass tidy (#34663)Martin Robinson2024-12-171-8/+42
| | | | | | The issue here is mainly that some of the lines are longer than 120 characters. Just add linebreak where appropriate. Signed-off-by: Martin Robinson <mrobinson@igalia.com>
* tidy: Use more `cargo-deny` features (#34447)Martin Robinson2024-12-131-53/+0
| | | | | | | | | | | | Instead of parsing the `Cargo.lock` file directly in `tidy.py`. Use `cargo-deny`, which we already use to detect unapproved licenses in the dependency chain to detect duplicate and banned crates. In addition, enable all other `cargo-deny` checks and add exceptions where necessary for them. This depends on the latest release of `cargo-deny` which depends on a recent verison of `rust`. Fixes #34393. Signed-off-by: Martin Robinson <mrobinson@igalia.com>
* tidy: Fix rustdoc warnings and add a tidy check for a common URL issue (#33366)Martin Robinson2024-09-081-0/+36
| | | | | | | | This change fixes all rustdoc errors and also adds a tidy check for a very common rustdoc URL issue. Eventually rustdoc warnings should likely cause the build to fail, but this catches those issues sooner in order to not waste so much developer time. Signed-off-by: Martin Robinson <mrobinson@igalia.com>
* Add cargo-deny to mach-tidy to check license compliance. (#32465)Jonathan Schwender2024-06-121-3/+3
| | | | | | | | | | | | | | | | | | | | | | | * Use cargo-deny to check license compliance. All licenses should be MPL-2.0 or weaker. * Run cargo-deny check licenses in mach tidy * fmt * Fix inverted boolean * Move cargo deny to tidy.py * Add quotes around license in error message * Integrate `cargo-deny` into tidy fully * Fix script tests --------- Co-authored-by: Martin Robinson <mrobinson@igalia.com>
* tidy: Fix WHATWG replacement links (#31449)Smitty2024-02-291-2/+2
|
* tidy: A few small improvements and fixes (#30941)Martin Robinson2024-01-021-13/+7
| | | | | | | | | | 1. Make the tidy output easier to follow 2. Integrate the WPT manifest cleanliness step into tidy itself and don't run it if nothing has changed in the WPT directory. 3. Fix an issue where Python test requirements were not installed, which could cause issues with some modules not being found. Fixes #30002.
* No extern crate test/tidy & better RUSTFLAGS handling in mach (#30328)Samson2023-09-101-1/+0
| | | | | | | | | | | * Remove unused extern crate rustflag * Remove tidy check alphabetical order of extern crates rustfmt's reorder_imports already does that * fix * better RUSTFLAGS handling
* Remove the YAML tidy code (#30004)Martin Robinson2023-07-181-15/+0
| | | | | | | | | This code was used to test buildbox_steps.yml, but Servo no longer uses buildbot, so this code is essentially unused. In addition, YAML + Cython 3 is causing issues on the CI. Fixes #30003 Signed-off-by: Martin Robinson <mrobinson@igalia.com>
* Combine all script tests into `test-scripts`Martin Robinson2023-06-291-7/+10
| | | | | | | | | | | Remove: - tidy self test support from `./mach test` - `./mach test-idl` Adds a `./mach test-scripts` command that is responsible for running all Python script tests. Run this during the CI to catch regressions in changes to scripts. The WebIDL tests are still *very* slow and there are from Gecko, so only run them when "-a" is passed meaning all tests.
* Convert tidy to a non-egg Python packageMartin Robinson2023-06-151-0/+267
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.