mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-09-05 00:56:39 +00:00
Meta: Fix unused variable warning in lint-ports.py
This went undetected because I broke CI so the Python linter didn't run.
This commit is contained in:
parent
52710c2672
commit
830f5c610d
Notes:
sideshowbarker
2024-07-17 10:46:09 +09:00
Author: https://github.com/BertalanD
Commit: 830f5c610d
Pull-request: https://github.com/SerenityOS/serenity/pull/14065
Reviewed-by: https://github.com/timschumi ✅
1 changed files with 0 additions and 3 deletions
|
@ -228,20 +228,17 @@ def check_descriptions_for_port_patches(patches):
|
||||||
|
|
||||||
patch_names = set(Path(x).stem for x in patch_files)
|
patch_names = set(Path(x).stem for x in patch_files)
|
||||||
|
|
||||||
patches_ok = True
|
|
||||||
for patch_name in patch_names:
|
for patch_name in patch_names:
|
||||||
if patch_name not in readme_contents:
|
if patch_name not in readme_contents:
|
||||||
print(f"Ports/{port}/patches/{patch_name}.patch does not appear to be described in"
|
print(f"Ports/{port}/patches/{patch_name}.patch does not appear to be described in"
|
||||||
" the corresponding ReadMe.md")
|
" the corresponding ReadMe.md")
|
||||||
all_good = False
|
all_good = False
|
||||||
patches_ok = False
|
|
||||||
|
|
||||||
for patch_name in readme_contents:
|
for patch_name in readme_contents:
|
||||||
if patch_name not in patch_names:
|
if patch_name not in patch_names:
|
||||||
print(f"Ports/{port}/patches/{patch_name}.patch is described in ReadMe.md, "
|
print(f"Ports/{port}/patches/{patch_name}.patch is described in ReadMe.md, "
|
||||||
"but does not actually exist")
|
"but does not actually exist")
|
||||||
all_good = False
|
all_good = False
|
||||||
patches_ok = False
|
|
||||||
|
|
||||||
return all_good
|
return all_good
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue