aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/tidy.py
Commit message (Collapse)AuthorAgeFilesLines
* Allow tidy to pass without support/crown/target (#34212)Samson2024-11-111-1/+1
| | | Signed-off-by: sagudev <16504129+sagudev@users.noreply.github.com>
* fix possible IndexError (#33750)Gae242024-10-101-1/+1
| | | Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
* tidy: Fix rustdoc warnings and add a tidy check for a common URL issue (#33366)Martin Robinson2024-09-081-1/+30
| | | | | | | | 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>
* mach: fix test-tidy to handle missing merge commit (#32551)Mukilan Thiyagarajan2024-06-191-0/+2
| | | | | | | | | This restores the behaviour prior to #32540 where the case where no merge commit being found (which happens on CI in forks because we do a shallow clone) is handled as if no changes where found. Fixes #32550. Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com>
* Use workspace definitions for all crates and update to the 2021 edition (#32544)Martin Robinson2024-06-181-0/+2
|
* mach: fix test-tidy to not skip `Cargo.lock` (#32540)Mukilan Thiyagarajan2024-06-181-9/+14
| | | | | | | | | | PR #32465 broke the lint because it initializes FileList with a file name (./Cargo.lock). This causes it to always return an empty list when the `only_changed_files` parameter is `False` since `os.walk` requires a directory and not a file. Fixes #32530. Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com>
* Add cargo-deny to mach-tidy to check license compliance. (#32465)Jonathan Schwender2024-06-121-37/+53
| | | | | | | | | | | | | | | | | | | | | | | * 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: Make sure to run WPT lints when `--all` is passed to tidy (#31874)Martin Robinson2024-03-261-1/+1
| | | This also fixes the checked in WPT manifest.
* tidy: Fix WHATWG replacement links (#31449)Smitty2024-02-291-2/+2
|
* Fix bugs in `mach test-tidy` (#31232)Mukilan Thiyagarajan2024-01-311-1/+1
| | | | | | | | | | | | | | | | | * mach: test-tidy should return 1 for failure Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com> * fix lint issues Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com> * mach: tidy should allow spec links with trailing text Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com> --------- Signed-off-by: Mukilan Thiyagarajan <mukilan@igalia.com>
* tidy: A few small improvements and fixes (#30941)Martin Robinson2024-01-021-147/+110
| | | | | | | | | | 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.
* Improve formatting of Python files (#30919)Taym Haddadi2023-12-241-15/+15
| | | Signed-off-by: Bentaimia Haddadi <haddadi.taym@gmail.com>
* No extern crate test/tidy & better RUSTFLAGS handling in mach (#30328)Samson2023-09-101-17/+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
* Rename ports/winit package to servoshell (#30163)Atbrakhi2023-08-281-3/+3
| | | | | * rename winit package to servoshell * revert previous changes and rename only package
* Remove the YAML tidy code (#30004)Martin Robinson2023-07-181-68/+4
| | | | | | | | | 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>
* Enable the GitHub merge queue (#29989)Martin Robinson2023-07-131-1/+1
| | | Signed-off-by: Martin Robinson <mrobinson@igalia.com>
* Update scripts to reflect new WPT directory nameMartin Robinson2023-06-261-4/+4
|
* Standalone Flake8 configsagudev2023-06-231-7/+1
|
* Convert tidy to a non-egg Python packageMartin Robinson2023-06-151-0/+1172
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.
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-07 02:50:38 +0000