Skip to content

Replace black with ruff for Python formatting and linting#75

Merged
michaelbarton merged 3 commits intomasterfrom
claude/create-pull-request-HAQjm
Mar 6, 2026
Merged

Replace black with ruff for Python formatting and linting#75
michaelbarton merged 3 commits intomasterfrom
claude/create-pull-request-HAQjm

Conversation

@michaelbarton
Copy link
Owner

Summary

  • Replaces black and isort with ruff for Python formatting and linting in the Makefile and cookiecutter template
  • Switches markdown formatting from prettier to mdformat for .md files
  • Applies ruff formatting fixes across Python files (unused import removal, line length, code style)

Test plan

  • Run make fmt_check to verify formatting and linting pass
  • Run make fmt to confirm formatting commands work correctly
  • Verify cookiecutter template generates projects with ruff configuration

https://claude.ai/code/session_0121xGdLzdAZVcxK4axAssQj

claude added 3 commits March 6, 2026 22:57
- Makefile: swap black for ruff format + ruff check --fix, use mdformat
  for markdown instead of prettier (prettier still handles YAML)
- Cookiecutter template: replace black/isort deps with ruff, add lint
  rule selection (E, W, F, I, UP, B, SIM, RUF) for pragmatic autofixes

https://claude.ai/code/session_01JYXb2pgjHPrxsu53G6SgLK
Ruff: reformat 2 files, remove unused import (parquet/parquet_to_csv.py)
mdformat: rewrap markdown to 80 chars

https://claude.ai/code/session_01JYXb2pgjHPrxsu53G6SgLK
The fmt_check target was missing --wrap 80 --number flags that the fmt
target uses, causing CI to fail on correctly formatted markdown files.

https://claude.ai/code/session_0121xGdLzdAZVcxK4axAssQj
@michaelbarton michaelbarton merged commit 3fdfd50 into master Mar 6, 2026
1 check passed
@michaelbarton michaelbarton deleted the claude/create-pull-request-HAQjm branch March 6, 2026 23:39
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.

2 participants