servers icon indicating copy to clipboard operation
servers copied to clipboard

Add PEP8 linting for Python servers

Open olaservo opened this issue 6 months ago • 1 comments

Is your feature request related to a problem? Please describe. The Python MCP servers in this repository (fetch, git, and time) currently include ruff as a dev dependency but lack standardized PEP8 linting configuration. This leads to inconsistent code style across the Python codebase and makes it difficult for contributors to ensure their code follows Python style guidelines.

Describe the solution you'd like Add standardized PEP8 linting configuration using ruff to all Python servers in the repository. This should include:

  • Consistent [tool.ruff] configuration sections in each Python server's pyproject.toml
  • Enable PEP8 style checks (E and W error codes)
  • Set up import sorting and formatting rules
  • Add linting commands to development workflows
  • Optionally integrate linting checks into CI/CD pipelines

Describe alternatives you've considered

  • Using flake8 + black + isort combination instead of ruff (but ruff is already included and provides all these features in one tool)
  • Using pylint for more comprehensive static analysis (but it's heavier and ruff covers PEP8 compliance well)
  • Adding pre-commit hooks with linting (could be done in addition to the main solution)

Additional context The repository already has ruff included in dev dependencies for all Python servers, so this enhancement would primarily involve adding configuration rather than new dependencies.

(Description generated by Claude and edited by me)

olaservo avatar Jun 25 '25 12:06 olaservo

@olaservo Also ran into this just now. I'd be happy to do a PR if there's nothing already in progress or blocking this?

chughtapan avatar Sep 01 '25 19:09 chughtapan