mirror of
https://github.com/servo/servo.git
synced 2025-08-03 04:30:10 +01:00
290a1696dc
Allow tidy to run custom project-specific lints <!-- Please describe your changes on the following line: --> Since tidy is a package, it shouldn't contain our WPT lint. This PR changes the file list generator (we already have) into an object, and allows tidy to run custom python lint scripts (specified in the config file) under the context of `LintRunner`. The errors are then chained into our mechanism. r? @frewsxcv (cc @jdm @edunham @aneeshusa and anyone else interested in reviewing it) --- <!-- 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 <!-- 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/14166) <!-- Reviewable:end -->
61 lines
2.1 KiB
TOML
61 lines
2.1 KiB
TOML
[configs]
|
|
skip-check-length = false
|
|
skip-check-licenses = false
|
|
check-ordered-json-keys = [
|
|
"./resources/prefs.json",
|
|
"./resources/package-prefs.json",
|
|
]
|
|
lint-scripts = [
|
|
"./python/servo/lints/wpt_lint.py",
|
|
]
|
|
|
|
[ignore]
|
|
# Ignored packages with duplicated versions
|
|
packages = ["bitflags", "lazy_static", "semver"]
|
|
# Files that are ignored for all tidy and lint checks.
|
|
files = [
|
|
# Generated and upstream code combined with our own. Could use cleanup
|
|
"./components/style/gecko_bindings/bindings.rs",
|
|
"./components/style/gecko_bindings/structs_debug.rs",
|
|
"./components/style/gecko_bindings/structs_release.rs",
|
|
"./components/style/gecko_string_cache/atom_macro.rs",
|
|
"./resources/hsts_preload.json",
|
|
"./tests/wpt/metadata/MANIFEST.json",
|
|
"./tests/wpt/metadata-css/MANIFEST.json",
|
|
"./components/script/dom/webidls/ForceTouchEvent.webidl",
|
|
"./support/android/openssl.sh",
|
|
# Ignore those files since the issues reported are on purpose
|
|
"./tests/html/bad-line-ends.html",
|
|
"./tests/unit/net/parsable_mime/text",
|
|
"./tests/wpt/mozilla/tests/css/fonts",
|
|
"./tests/wpt/mozilla/tests/css/pre_with_tab.html",
|
|
# FIXME(pcwalton, #11679): This is a workaround for a tidy error on the quoted string
|
|
# `"__TEXT,_info_plist"` inside an attribute.
|
|
"./components/servo/platform/macos/mod.rs",
|
|
]
|
|
# Directories that are ignored for the non-WPT tidy check.
|
|
directories = [
|
|
# Upstream
|
|
"./support/android/apk",
|
|
"./tests/wpt/css-tests",
|
|
"./tests/wpt/harness",
|
|
"./tests/wpt/update",
|
|
"./tests/wpt/web-platform-tests",
|
|
"./tests/wpt/mozilla/tests/mozilla/referrer-policy",
|
|
"./tests/wpt/sync",
|
|
"./tests/wpt/sync_css",
|
|
"./python/mach",
|
|
"./python/tidy/servo_tidy_tests",
|
|
"./components/script/dom/bindings/codegen/parser",
|
|
"./components/script/dom/bindings/codegen/ply",
|
|
"./python/_virtualenv",
|
|
# Generated and upstream code combined with our own. Could use cleanup
|
|
"./target",
|
|
"./ports/cef",
|
|
"./components/style/gecko_bindings/nsstring_vendor/",
|
|
]
|
|
|
|
# Directories that are checked for correct file extension
|
|
[check_ext]
|
|
# directory, list of expected file extensions
|
|
"components/script/dom/webidls" = [".webidl"]
|