Skip to content

chore: Configure ruff in its own file, add precommit hooks#72

Merged
effigies merged 10 commits intomainfrom
add-ruff-and-precommit-hook
Mar 13, 2026
Merged

chore: Configure ruff in its own file, add precommit hooks#72
effigies merged 10 commits intomainfrom
add-ruff-and-precommit-hook

Conversation

@bendhouseart
Copy link
Copy Markdown
Contributor

Just like the title states.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.89%. Comparing base (878d57d) to head (f268033).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   90.89%   90.89%           
=======================================
  Files          13       13           
  Lines         846      846           
  Branches      124       91   -33     
=======================================
  Hits          769      769           
  Misses         47       47           
  Partials       30       30           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

bendhouseart and others added 5 commits March 12, 2026 23:39
=== Do not change lines below ===
{
 "chain": [],
 "cmd": "uv lock",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [
  "pyproject.toml",
  "uv.lock"
 ],
 "outputs": [
  "uv.lock"
 ],
 "pwd": "."
}
^^^ Do not change lines above ^^^
=== Do not change lines below ===
{
 "chain": [],
 "cmd": "bash -c 'ruff check --fix && ruff format'",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@effigies effigies force-pushed the add-ruff-and-precommit-hook branch from b13d4a3 to a507fa9 Compare March 13, 2026 03:40
@effigies
Copy link
Copy Markdown
Contributor

I suppose I should have checked that we didn't already have a ruff config in pyproject.toml before reviewing this. I've moved the config over to ruff.toml for the sake of copyability. I also commented our extend-select and ignores, so we can choose whether to adopt these in other projects.

@effigies effigies changed the title Add ruff and precommit hook chore: Configure ruff in its own file, add precommit hooks Mar 13, 2026
@effigies effigies merged commit 1963ba3 into main Mar 13, 2026
40 of 41 checks passed
@effigies effigies deleted the add-ruff-and-precommit-hook branch March 13, 2026 12:09
@bendhouseart
Copy link
Copy Markdown
Contributor Author

I suppose I should have checked that we didn't already have a ruff config in pyproject.toml before reviewing this. I've moved the config over to ruff.toml for the sake of copyability. I also commented our extend-select and ignores, so we can choose whether to adopt these in other projects.

I looked there first before I started this endeavor, but forgot as well ;)

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