aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy
Commit message (Collapse)AuthorAgeFilesLines
* 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
|
* Auto merge of #12781 - UK992:tidycheck-rebased, r=Wafflespeanutbors-servo2016-08-142-103/+61
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | Improve tidy's license validation logic Rebased and fixed https://github.com/servo/servo/pull/10721, which is inactive for months. Fixes https://github.com/servo/servo/issues/10716 r? @larsbergstrom or @edunham <!-- 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/12781) <!-- Reviewable:end -->
| * Add support colored text in Windows ConsoleUK9922016-08-131-0/+3
| |
| * Check for blank line after shebang and grab actual comment block instead of ↵UK9922016-08-122-8/+13
| | | | | | | | hard-coded one
| * Improve tidy's license validation logicedunham2016-08-122-103/+53
| | | | | | | | | | | | | | | | | | | | | | fixes https://github.com/servo/servo/issues/10716 I took the lazy way out and hardcoded the size of block we examine for licenses. fixes https://github.com/servo/servo/issues/10719 Includes tests for new functionality.
* | Update ordered-float.Ms2ger2016-08-111-1/+1
| |
* | Issue #12421: tidy should also check .html filesSimon Martin2016-08-091-4/+9
| |
* | Prefer length and percentage for word spacingRavi Shankar2016-08-091-1/+1
|/
* Add lint for `[` instead of `[[` in shells scriptsAneesh Agrawal2016-08-051-0/+3
|
* Add lint to ensure substitutions use the full formAneesh Agrawal2016-08-051-0/+7
| | | | | Check that any variable substitutions use the full ${VAR} form, not just $VAR (but don't check for quoting yet).
* Add lint for backticks in shell scriptsAneesh Agrawal2016-08-051-10/+17
| | | | | | The "$(some_command arg1 arg2)" form is preferred to the `some_command arg1 arg2` form because it nests unambiguously. Add a lint for this to tidy.
* Auto merge of #12295 - jimberlage:12158/shell-linting, r=aneeshusabors-servo2016-08-042-3/+47
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add linting for shell scripts <!-- Please describe your changes on the following line: --> This changes tidy to check shell scripts for the proper shebang and options. It does not check that variables are formatted correctly. It also adds a check for the MPL 2.0 license in shell scripts. --- <!-- 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 #12158 (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="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12295) <!-- Reviewable:end -->
| * Add linting for shell scriptsJim Berlage2016-07-222-3/+47
| | | | | | | | | | | | This changes tidy to check shell scripts for the proper shebang and options. It does not check that variables are formatted correctly. It also adds a check for the MPL 2.0 license in shell scripts.
* | Add python3 licence check templateShing Lyu2016-07-291-0/+8
|/
* Auto merge of #12511 - asajeffrey:test-tidy-dep-licenses, r=larsbergstrombors-servo2016-07-212-2/+61
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added test-tidy for dependent licenses. <!-- Please describe your changes on the following line: --> Add a test-tidy lint for dependency licenses. Note that #12507 should land first. --- <!-- 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 do not require tests because we don't test our lints <!-- 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/12511) <!-- Reviewable:end -->
| * Added test-tidy for dependent licenses.Alan Jeffrey2016-07-212-2/+61
| |
* | Add the append method for the Headers API for the Fetch APIJeena Lee2016-07-191-0/+1
|/ | | | This commit will add the append method and associated helper functions, and introduce any necessary changes for it.
* Allow MIT/Apache-2.0 license as well as MPL-2.0.Alan Jeffrey2016-07-142-5/+12
|
* All our Cargo.toml files should contain an MPL-2.0 license field.Alan Jeffrey2016-07-141-0/+4
|
* [tidy] check for duplicate keys in .json filesOwen Coutts2016-07-081-1/+13
|
* tidy command now ignores files in subdirectories of ignored directoriesNikhil Shagrithaya2016-07-071-1/+1
|
* stylo: Move all binding-generator code to a python script.Emilio Cobos Álvarez2016-07-051-2/+0
| | | | | | | | | | | | | | | | | | | | This not only makes us more consistent with the rest of the codebase but also: * Makes us repeat less code like common flags and all that stuff. * Reduces the noise of the build: You only get the output of the commands on failure or when you pass the -v flag. * Makes you able to select a single kind of build or multiple in the same place. I've basically kept the regen.sh script because of the LIBCLANG_PATH checks, but at least from Linux I don't need them anymore. Also, that logic could be moved to the new script. The whole point of this isn't only making it prettier and easier to use, but also allowing me to write more complex logic in the binding generator scripts, that I will probably need to integrate the DOM enum types we need for animations and such easily (can't be just an include, because that pulls in another header with the same name bringing a lot of DOM and IDL churn).
* Add style check, test, and code fixes for an else brace check.Travis Dean2016-07-041-0/+1
|
* Update web-platform-tests to revision 346d5b51a122f7bb1c7747064499ef281a0200f7Ms2ger2016-06-241-1/+1
|
* Use our copy of RefCell for style data.Simon Sapin2016-06-231-1/+12
| | | | | This allows removing `#![feature(as_unsafe_cell)]` in geckolib and make progress towards #11815.
* Removed unused function and minor cleanupRavi Shankar2016-06-161-76/+63
|
* Auto merge of #11621 - h4xr:mach_fix, r=Wafflespeanutbors-servo2016-06-141-2/+3
|\ | | | | | | | | | | | | | | | | | | | | | | | | Make mach test-tidy consider ignored dirs Made changes so that mach test-tidy considers the ignored directories Fixes #11386 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11621) <!-- Reviewable:end -->
| * Make mach consider ignored dirsSaurabh Badhwar2016-06-151-2/+3
| | | | | | | | Address indentation changes
* | servo: Add an `Info.plist` on the Mac and opt into integrated graphics.Patrick Walton2016-06-081-0/+3
| | | | | | | | | | | | | | | | | | | | | | Discrete GPUs cause power use problems and tend to perform badly with WebRender. See: * https://developer.apple.com/library/mac/qa/qa1734/_index.html * https://reverse.put.as/2013/05/28/gimmedebugah-how-to-embedded-a-info-plist-into-arbitrary-binaries/
* | Auto merge of #11190 - mrmiywj:Root-tidy-checker, r=jdmbors-servo2016-06-051-0/+2
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add &Root<T> checker 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 #11137 (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="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11190) <!-- Reviewable:end -->
| * add &Root<T> checkermrmiywj2016-06-051-0/+2
| | | | | | | | | | | | update rust_tidy tests update rust_tidy.rs and test_tidy.py
* | Generate debug and release Gecko bindings.Cameron McCormack2016-06-031-1/+2
| |
* | implement related sw interface and register methodRahul Sharma2016-06-021-0/+1
| |
* | Remove the gonk portLars Bergstrom2016-05-271-1/+0
| |
* | Warn about empty lines following an open brace.Josh Matthews2016-05-271-2/+7
| |
* | Report use statements that use {} with only one entryCullen Rhodes2016-05-271-0/+2
| |
* | Add mozilla tests for referrer policy in iframe without postmessageRebecca2016-05-261-0/+1
| | | | | | | | Can be removed and web-platform-tests not ignored with postmessage support
* | Add --all for test-tidy, with backwards-compat with --fasterMitchell Hentges2016-05-241-5/+5
| |
* | Auto merge of #11242 - bholley:gecko_atoms, r=emiliobors-servo2016-05-181-0/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add basic support for Gecko atoms This is a rough initial implementation of gecko atoms. This allows us to get rid of the custom rust-selectors branch we use to build stylo. The gecko changes are at https://bugzilla.mozilla.org/show_bug.cgi?id=1273771 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11242) <!-- Reviewable:end -->
| * | Add string_cache override for geckolib based on gecko atoms.Bobby Holley2016-05-181-0/+1
| | | | | | | | | | | | The work in these patches is based on Ms2ger's gecko-atom crate.
* | | Support test-tidy --faster with moved files.Ms2ger2016-05-181-0/+3
|/ /
* / Disallow bitflags dupes againAnthony Ramine2016-05-181-1/+1
|/
* Auto merge of #10713 - aeischeid:master, r=jdmbors-servo2016-05-111-2/+3
|\ | | | | | | | | | | | | | | | | | | | | | | | | add tidy test for space after ":" in stucts Addresses issue #10702 the tidy self-test doesn't seem to catch the `member_name:"Foo"` line like it should. the regex follows similar pattern in the file and works in my regex tester tool, so not really sure what is going on there. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10713) <!-- Reviewable:end -->
| * add tidy test for 'missing space after :' in stuctsaeischeid2016-05-111-2/+3
| |
generated by cgit v1.2.3 (git 2.25.1) at 2025-05-06 23:33:39 +0000