Skip to content

Set up Ruff, pre-commit hooks, and dev tooling - #55

Merged
JoseVelazcoH merged 1 commit into
developfrom
issue-14-dev-tooling
Jun 30, 2026
Merged

JoseVelazcoH merged 1 commit into
developfrom
issue-14-dev-tooling

Conversation

@zamax14

@zamax14 zamax14 commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds Ruff and pre-commit to the development extra.
  • Adds Ruff config based on the ETL_SIEEJ setup, scoped to unused imports and redefinitions.
  • Adds CI for Ruff check and pytest across Python 3.10, 3.11, and 3.12.
  • Makes __version__ resolve from package metadata with a fallback for local source checkouts.
  • Removes only the unused imports needed for the new Ruff check to pass; this avoids global formatting churn while other feature issues are being reviewed.

Tests

  • .venv/bin/ruff check .
  • .venv/bin/python -m pytest
  • .venv/bin/pre-commit run ruff --all-files
  • .venv/bin/pre-commit run trailing-whitespace --all-files
  • .venv/bin/pre-commit run end-of-file-fixer --all-files
  • .venv/bin/pre-commit run check-yaml --all-files
  • .venv/bin/pre-commit run check-merge-conflict --all-files

Notes

  • ruff-format is installed as a pre-commit hook, but this PR intentionally does not run a repository-wide format to avoid conflicts with the other issue branches being reviewed in parallel.

@zamax14 zamax14 self-assigned this Jun 29, 2026
@zamax14 zamax14 linked an issue Jun 29, 2026 that may be closed by this pull request
@zamax14
zamax14 marked this pull request as ready for review June 29, 2026 08:38
@zamax14
zamax14 requested a review from JoseVelazcoH June 29, 2026 08:38
@JoseVelazcoH
JoseVelazcoH merged commit d0bee0e into develop Jun 30, 2026
3 checks passed
@JoseVelazcoH
JoseVelazcoH deleted the issue-14-dev-tooling branch June 30, 2026 21:09
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.

Set up Ruff, pre-commit hooks, and dev tooling

2 participants