diff --git a/.github/workflows/reusable_checks.yml b/.github/workflows/reusable_checks.yml index 7ab9d1503895a..a8f8f38040e50 100644 --- a/.github/workflows/reusable_checks.yml +++ b/.github/workflows/reusable_checks.yml @@ -272,6 +272,10 @@ jobs: run: | ./scripts/lint.py + - name: Check for zombie TODOs + run: | + ./scripts/zombie_todos.py --token ${{ secrets.GITHUB_TOKEN }} + - name: Check for too large files run: | ./scripts/ci/check_large_files.sh diff --git a/scripts/ci/requirements.txt b/scripts/ci/requirements.txt index 061cbed7d472e..39f2fdf7e52c1 100644 --- a/scripts/ci/requirements.txt +++ b/scripts/ci/requirements.txt @@ -4,11 +4,14 @@ -r ../../rerun_py/requirements-lint.txt Jinja2==3.1.2 +Pillow==10.0.0 PyGithub==1.59.0 + +aiohttp==3.8.5 colorama==0.4.6 +gitignore-parser==0.1.4 google-cloud-storage==2.9.0 packaging==23.1 +python-frontmatter==1.0.0 requests>=2.31,<3 tomlkit==0.11.8 -python-frontmatter==1.0.0 -Pillow==10.0.0 diff --git a/scripts/lint.py b/scripts/lint.py index 0e18c2e44ab98..2a6dd8591ed65 100755 --- a/scripts/lint.py +++ b/scripts/lint.py @@ -628,6 +628,7 @@ def main() -> None: "./scripts/zombie_todos.py", "./web_viewer/re_viewer.js", # auto-generated by wasm_bindgen "./web_viewer/re_viewer_debug.js", # auto-generated by wasm_bindgen + ".github/workflows/reusable_checks.yml", # zombie TODO hunting job } should_ignore = parse_gitignore(".gitignore") # TODO(emilk): parse all .gitignore files, not just top-level diff --git a/scripts/requirements-dev.txt b/scripts/requirements-dev.txt index bef6b944e98f6..4be81c877cfd9 100644 --- a/scripts/requirements-dev.txt +++ b/scripts/requirements-dev.txt @@ -12,4 +12,3 @@ tqdm requests gitignore_parser # handle .gitignore python-frontmatter==1.0.0 -aiohttp