Light Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix All Pre-commit Hook Linting Errors #844

Open
Open
Fix All Pre-commit Hook Linting Errors#844
Assignees

Description

Objective

Run the pre-commit hooks, identify all linting errors, and fix every single one of them. Do NOT make any changes outside of what is needed to fix the linting errors -- no refactoring, no feature changes, no config changes.

Step 1: Setup

pip install pre-commit
npm ci
cd frontend && npm ci && cd ..

Step 2: Identify Issues

pre-commit run --all-files

Read the output carefully. Note every failing hook, its errors, affected files, and line numbers.

Step 3: Fix All Errors

Fix every error reported. For semgrep security findings where the code is safe (e.g., hardcoded template paths, internally guarded endpoints), suppress with an inline # nosemgrep: comment after verifying the code is genuinely safe. For all other linting errors, fix the underlying code issue.

Step 4: Verify

Re-run until clean:

pre-commit run --all-files

Every hook must show Passed or Skipped. Zero hooks should show Failed.

Constraints

  • Do NOT modify any config files (.pre-commit-config.yaml, pyproject.toml, .semgrepignore, .flake8, eslint.config.js, package.json, etc.)
  • Only modify source files to fix the reported errors
  • Do not break any existing functionality

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions