Skip to content

Conversation

@codingjoe
Copy link
Owner

No description provided.

@codingjoe codingjoe self-assigned this Nov 3, 2025
@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.41%. Comparing base (746ccaa) to head (fa797a3).
⚠️ Report is 111 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #165      +/-   ##
==========================================
- Coverage   99.69%   99.41%   -0.29%     
==========================================
  Files          21       21              
  Lines         332      343      +11     
==========================================
+ Hits          331      341      +10     
- Misses          1        2       +1     
Flag Coverage Δ
python-3.10 93.00% <ø> (?)
python-3.11 93.00% <ø> (?)
python-3.12 93.00% <ø> (?)
python-3.13 93.00% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI review requested due to automatic review settings November 3, 2025 10:40
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the project's development workflow by migrating from manual CI-based linting to pre-commit hooks. The changes remove the separate lint job from CI and consolidate linting tools into a pre-commit configuration, while also cleaning up unnecessary files and formatting inconsistencies.

  • Removes the lint optional dependency group and the dedicated lint job from CI workflows
  • Introduces a comprehensive .pre-commit-config.yaml with hooks for code formatting, linting, and quality checks
  • Removes obsolete files (MANIFEST.in, .bandit) and cleans up whitespace formatting across configuration files

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Removes the lint optional dependency group containing ruff, pyupgrade, and django-upgrade
MANIFEST.in Deletes the entire file (no longer needed)
.readthedocs.yaml Removes blank lines for cleaner formatting
.pre-commit-config.yaml Adds comprehensive pre-commit configuration with hooks for various code quality checks
.github/workflows/release.yml Removes blank lines for cleaner formatting
.github/workflows/ci.yml Removes the lint job from CI and updates job dependencies to exclude it
.github/dependabot.yml Fixes indentation to be consistent
.bandit Deletes the entire file (no longer needed)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codingjoe codingjoe merged commit 08597e0 into main Nov 3, 2025
21 of 22 checks passed
@codingjoe codingjoe deleted the pre-commit branch November 3, 2025 10:44
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.

2 participants