Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Developer QOL #1174

Merged
merged 5 commits into from
Feb 10, 2025
Merged

Developer QOL #1174

merged 5 commits into from
Feb 10, 2025

Conversation

kingbuzzman
Copy link
Contributor

@kingbuzzman kingbuzzman commented Feb 10, 2025

  • This update addresses two common pain points when developing on github.com's browser editor:
    1. 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.
    2. 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

@kingbuzzman kingbuzzman marked this pull request as ready for review February 10, 2025 09:29
Copy link
Member

@bluetech bluetech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@bluetech bluetech merged commit c3018d6 into pytest-dev:main Feb 10, 2025
19 checks passed
@kingbuzzman kingbuzzman deleted the patch-2 branch February 10, 2025 12:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants