Conversation
AGENTS.md
Outdated
|
|
||
| ## Type Checking | ||
|
|
||
| - Always run mypy from the main `.venv`. If there is no `.venv`, create it and install `pip install -r requirements-lint.txt -r requirements-test.txt` |
There was a problem hiding this comment.
We should create a tox env that only runs mypy imo, feel like it's easy for a virtual env to be polluted.
There was a problem hiding this comment.
I like the idea. Added ruff and mypy as tox envs and tweaked the file.
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. Bug Fixes 🐛
Documentation 📚
🤖 This preview updates automatically when you update the PR. |
Codecov Results 📊✅ 1707 passed | ⏭️ 184 skipped | Total: 1891 | Pass Rate: 90.27% | Execution Time: 2m 21s All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 11872 uncovered lines. Generated by Codecov Action |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
sl0thentr0py
left a comment
There was a problem hiding this comment.
nice, do we want to keep .cursor or remove it now?
Description
Add a tailored AGENTS.md.
Tested with Claude fixing a small issue in the Celery integration. It correctly identified where to make the change, where to add the tests, and how to run the correct test suite.
Issues
Closes #5576
Reminders
tox -e linters.feat:,fix:,ref:,meta:)