aboutsummaryrefslogtreecommitdiffstats
path: root/python/tidy/servo_tidy
Commit message (Collapse)AuthorAgeFilesLines
...
* Add the option in servo-tidy to not check for alphabetical ordering.Nicolas Silva2018-03-061-6/+8
|
* Change bad namingsTimur Borkhodoev2018-02-021-4/+4
|
* Add self-test and modify tidy to accomodate the testTimur Borkhodoev2018-02-011-2/+2
|
* Add proper formatting and ignore _mozillaTimur Borkhodoev2018-02-011-6/+10
|
* Prints errors if wpt manifest directoriries are not presentTimur Borkhodoev2018-02-011-2/+41
|
* Update buildbot_steps lint to handle env variablesAneesh Agrawal2018-01-081-11/+22
| | | | | | | | | | 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-2/+9
| | | | Resolves: #19306
* Fix a typo in tidy.pyAnthony Ramine2017-11-131-1/+1
|
* Auto merge of #19070 - jdm:csstests, r=metajackbors-servo2017-11-021-1/+1
|\ | | | | | | | | | | | | | | | | | | | | Enable CSS tests. Do not merge this; I'm looking into what it takes to stop running the test-css jobs. <!-- 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/19070) <!-- Reviewable:end -->
| * Account for changes to Cargo.lock format.Josh Matthews2017-11-021-1/+1
| |
* | Update WebGL conformance suite 1.0.3 from upstreamImanol Fernandez2017-10-311-1/+1
|/
* Stop relying on linking details of std’s default allocatorSimon Sapin2017-10-191-0/+4
| | | | | | | | | | | | We’ve been bitten before by symbol names changing: https://github.com/servo/heapsize/pull/46 and upstream is planning to stop using jemalloc by default: https://github.com/rust-lang/rust/issues/33082#issuecomment-309781465 So use the (relatively) new `#[global_allocator]` attribute to explicitly select the system allocator on Windows and jemalloc (now in an external crate) on other platforms. This choice matches current defaults.
* Fix commonmark Markdown warnings in docs, part 1Matt Brubeck2017-10-171-1/+1
| | | | | | | | 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.
* Update domparsing spec links to not point at WHATWGKafji2017-10-141-1/+0
| | | | | | - Change domparsing spec link from domparsing.spec.whatwg.org to w3c.github.io/DOM-Parsing - Remove from tidy since it's already covered by w3c.github.io - Update test manifest
* Rename DOMRefCell<T> to DomRefCell<T>Anthony Ramine2017-09-261-3/+3
| | | | | | | | 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-261-4/+4
|
* Rename JS<T> to Dom<T>Anthony Ramine2017-09-261-3/+3
|
* Fix some messages in the tidy scriptAnthony Ramine2017-09-261-3/+3
| | | | It's MutJS<T>, not MutJS<JS<T>>.
* Make tidy aware of Rust multiline stringsMichael Droettboom2017-09-211-11/+18
| | | | | | 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.
* Auto merge of #18179 - davidcl:master, r=jdmbors-servo2017-08-231-0/+13
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatically verify that derive() lists are alphabetically ordered #… <!-- Please describe your changes on the following line: --> Automatically verify that derive() lists are alphabetically ordered #18172 --- <!-- 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 #18172 (github issue number if applicable). <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- 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/18179) <!-- Reviewable:end -->
| * Automatically verify that derive() lists are alphabetically orderedClément DAVID2017-08-231-0/+13
| |
* | Add paint metrics to Performance Timeline APIFernando Jiménez Moreno2017-08-231-0/+1
|/
* tidy: add extra space after keyword checkPavel Sergeev2017-07-201-0/+6
|
* Upgrade to the latest version of WebRenderMartin Robinson2017-07-131-1/+1
|
* Count number of braces on a line in .Alan Jeffrey2017-06-071-7/+6
|
* Don't lint commit messages anymoreAnthony Ramine2017-05-231-31/+1
| | | | This lint cannot cope with rollup merges and didn't do its job correctly anyway
* Auto merge of #16893 - MortimerGoro:webgl_extensions, r=emiliobors-servo2017-05-191-0/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement WebGL extensions <!-- Please describe your changes on the following line: --> This PR provides the base for implementing WebGL extensions. It comes with the following ones already implemented and passing all related WebGL conformance tests: - OES_texture_float - OES_texture_float_linear - OES_texture_half_float - OES_texture_half_float_linear - OES_vertex_array_object I'll submit other extensions like compressed textures in a separate PR to ease the review process. I included the 5 extensions in this PR because it's easier to show/review how the WebGL extension base works. To pass all OES_texture_float_xxx tests I had to add some missing format conversions in WebGLRenderingContext.rs --- <!-- 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 - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes OR - [ ] These changes do not require tests because _____ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- 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/16893) <!-- Reviewable:end -->
| * Implement WebGL extensions.Imanol Fernandez2017-05-181-0/+1
| |
* | Make some attr values case-insensitive in selectorsSimon Sapin2017-05-181-0/+10
|/ | | | https://bugzilla.mozilla.org/show_bug.cgi?id=1363531
* Implemented Houdini worklets.Alan Jeffrey2017-05-171-0/+1
|
* Make tidy check that opening and closing braces that begin a line do so with ↵coalman2017-04-181-3/+12
| | | | proper alignment.
* Check for merge commits with tidySadman Kazi2017-04-131-4/+17
| | | | | | Increase merge commit check range under travis Add comment on why travis workaround is necessary
* Tidy: Prefer monkey patching sys.path over addsitedir (and some cleanup)Ravi Shankar2017-03-201-19/+20
|
* added tidy check for -> ()lucantrop2017-03-131-0/+2
|
* Rewrite the ban-type lint in PythonJefry Lagrange2017-02-251-0/+6
| | | | | | 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-231-0/+4
| | | | | | | | | | | | | | | | | | | | | 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: Ignore [replace]’d packages in the duplicate package lint.Simon Sapin2017-02-141-0/+2
|
* Tidy checks for styloManish Goregaokar2017-02-081-15/+26
|
* Tidy: Report invalid listed ignored directories and filesUK9922017-01-181-5/+34
|
* Moved tidy's BLOCKED_PACKAGES to the config fileMatthias Devlamynck2017-01-141-23/+6
|
* Rust tidy: disallow assignment = in the beginning of lineJure Podgoršek2017-01-061-0/+2
|
* Added test-tidy check for blocked packages.Alan Jeffrey2017-01-051-0/+35
|
* Fix tidy json alphabetical order checkUK9922016-12-291-1/+1
|
* Tidy: Check Cargo.lock for packages with same version and different sourcesUK9922016-12-261-42/+23
|
* Tidy: Simplify path normalizationUK9922016-12-261-15/+11
|
* Add a tidy check for problematic match cases in script_thread.rsJosh Matthews2016-12-161-0/+3
|
* Auto merge of #14051 - birryree:tidy-check-buildbot-steps, r=aneeshusabors-servo2016-12-081-4/+58
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-081-4/+58
| | | | | | | | | | | | | | | | | | 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
* | Implement tidy commit message testSijmen Schoon2016-12-031-1/+19
| | | | | | Tests for work in progress commits.
* | Move to Cargo workspacesAnthony Ramine2016-11-281-0/+2
| |
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-29 02:44:12 +0000