diff options
| author | Martin Robinson <mrobinson@igalia.com> | 2024-01-02 07:14:51 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-01-02 06:14:51 +0000 |
| commit | 516cc2cbca55a1d1505a494875d0c2022ac314c2 (patch) | |
| tree | a62f5c27b7027a10b8a2ab6c4b167f78df9cdaeb /python/wpt | |
| parent | f58541e65263daf60d3f5c3f8ad075b09d25223b (diff) | |
| download | servo-516cc2cbca55a1d1505a494875d0c2022ac314c2.tar.gz servo-516cc2cbca55a1d1505a494875d0c2022ac314c2.zip | |
tidy: A few small improvements and fixes (#30941)
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.
Diffstat (limited to 'python/wpt')
| -rw-r--r-- | python/wpt/manifestupdate.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/python/wpt/manifestupdate.py b/python/wpt/manifestupdate.py index c1428aa2cca..b7e3bd9fb1a 100644 --- a/python/wpt/manifestupdate.py +++ b/python/wpt/manifestupdate.py @@ -31,8 +31,9 @@ def create_parser(): return p -def update(check_clean=True, rebuild=False, **kwargs): - logger = wptlogging.setup(kwargs, {"mach": sys.stdout}) +def update(check_clean=True, rebuild=False, logger=None, **kwargs): + if not logger: + logger = wptlogging.setup(kwargs, {"mach": sys.stdout}) kwargs = {"config": os.path.join(WPT_PATH, "config.ini"), "product": "servo", "manifest_path": os.path.join(WPT_PATH, "meta"), |