Skip to content

Commit

Permalink
Developer QOL (#1174)
Browse files Browse the repository at this point in the history
This update addresses two common pain points when developing on github.com's browser editor:

Delayed CI Testing: Small changes often accumulate, causing the CI to wait for all previous commits before testing your current commit. This delay can be inefficient. Suggestion: Cancel previous CI runs when pushing new commits to speed up the feedback loop.

Lack of Fix Suggestions for ruff Issues: When ruff flags an issue, it highlights the problematic code block but doesn't suggest how to fix it. By adding the --diff option, you can see the differences and get guidance on correcting the issue.

All this came about while working on #1170
  • Loading branch information
kingbuzzman authored Feb 10, 2025
1 parent 120c2a5 commit c3018d6
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ on:
branches:
- main

concurrency:
group: ci-main-${{ github.ref }}
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}

env:
PYTEST_ADDOPTS: "--color=yes"

Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ deps =
ruff==0.9.5
mypy==1.15.0
commands =
ruff check {posargs:pytest_django pytest_django_test tests}
ruff check --diff {posargs:pytest_django pytest_django_test tests}
ruff format --quiet --diff {posargs:pytest_django pytest_django_test tests}
mypy {posargs:pytest_django pytest_django_test tests}

Expand Down

0 comments on commit c3018d6

Please # to comment.