Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the project’s dependency/tooling setup and applies repository-wide formatting changes, including raising the minimum supported Python version to 3.12 and adjusting CI accordingly.
Changes:
- Bump minimum Python to 3.12 and refresh locked dependencies (e.g., Pillow, rapidpro-python, smartmin, ruff).
- Replace Black with
ruff formatin local checks and remove Black from dev dependencies/config. - Apply formatting-only edits across multiple modules/tests and update CI test matrix versions.
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
uv.lock |
Regenerates lockfile for Python 3.12 baseline and updated dependency set. |
pyproject.toml |
Raises requires-python to 3.12, updates deps, removes Black, configures Ruff as formatter/linter. |
code_check.py |
Switches formatting check from Black to ruff format --check. |
.github/workflows/ci.yml |
Updates Python/Django test matrix and release job Python version. |
dash/orgs/views.py |
Formatting changes to messages and removal of blank lines. |
dash/orgs/models.py |
Formatting changes (string joining and field declaration wrapping). |
dash/dashblocks/models.py |
Formatting changes to help_text strings and field declarations. |
dash/stories/models.py |
Collapses concatenated help_text string into a single literal. |
dash/tags/models.py |
Removes an extra blank line in model definition. |
dash/categories/models.py |
Collapses multi-line field definition to a single line. |
test_runner/settings.py |
Collapses concatenated help_text strings into single literals. |
test_runner/tests.py |
Removes a stray blank line inside a loop. |
test_runner/testapp/tests.py |
Collapses multi-line asserts/queries to single lines (formatting-only). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
norkans7
force-pushed
the
update-deps-formatting
branch
from
7fc7f36 to
34061cf
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #236 +/- ##
=======================================
Coverage 93.32% 93.32%
=======================================
Files 29 29
Lines 1843 1843
=======================================
Hits 1720 1720
Misses 123 123 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "description", | ||
| models.TextField( | ||
| help_text="A description of where this content type is used on the site and how it will be dsiplayed", | ||
| help_text="A description of where this content type is used on the site and how it will be displayed", |
There was a problem hiding this comment.
This migration edits the original 0001 migration to change field help_text. Since there are many subsequent migrations in this app, rewriting an already-applied migration can create churn/merge conflicts and makes historical migration state harder to reason about. Prefer leaving 0001 unchanged and adding a new migration (or regenerating via makemigrations) that updates the field options.
| help_text="A description of where this content type is used on the site and how it will be displayed", |
No description provided.