Skip to content

chore: add ruff as linter#62

Merged
abubnalitic-nbl merged 1 commit into
mainfrom
chore/add-ruff-linter
Jan 8, 2026
Merged

chore: add ruff as linter#62
abubnalitic-nbl merged 1 commit into
mainfrom
chore/add-ruff-linter

Conversation

@abubnalitic-nbl

Copy link
Copy Markdown
Contributor

No description provided.

@abubnalitic-nbl abubnalitic-nbl self-assigned this Jan 8, 2026
@abubnalitic-nbl abubnalitic-nbl merged commit b1d1fe1 into main Jan 8, 2026
4 checks passed
@abubnalitic-nbl abubnalitic-nbl deleted the chore/add-ruff-linter branch January 8, 2026 15:47
@mhajder

mhajder commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

This should be reverted! ruff is added to prod dependencies and not dev! We don't need ruff for running package, also there is a lot of format errors, you can check that using ruff format --check. Probably it would be good idea to add pre-commit.

My example pre-commit config.

repos:
  - repo: https://github.com/astral-sh/ruff-pre-commit
    rev: v0.14.10
    hooks:
      - id: ruff-check
        args: [--fix]
      - id: ruff-format

  - repo: local
    hooks:
      - id: pytest
        name: pytest
        entry: uv run pytest
        language: system
        types: [python]
        pass_filenames: false
        always_run: true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants