Skip to content

Conversation

@GabDug
Copy link
Owner

@GabDug GabDug commented Jan 27, 2026

Doc & tests update only.


Note

Docs/tests for prek config support; CI matrix updates

  • README: note prek config support (preserve prek-specific keys) and allow pre-commit-config-file to be .pre-commit-config.yml; update supported versions to Poetry 2.3+ and PDM 2.26+.
  • Tests: add test_prek_config_support to ensure updating repo versions preserves prek-specific keys/structure.
  • CI/tox: add Poetry 2.3 across Python versions, tweak coverage envs, normalize VCS deps, add hishel<1.0 pins for py312 with specific PDM versions, update GH env matrix accordingly.
  • Tooling: bump .mise.toml PDM to 2.26.

Written by Cursor Bugbot for commit 32c01cb. This will update automatically on new commits. Configure here.

@GabDug GabDug enabled auto-merge (rebase) January 27, 2026 20:56
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

for call in handle.writelines.call_args_list:
written_lines.extend(call[0][0])

written_content = "".join(written_lines)
Copy link

Choose a reason for hiding this comment

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

Test uses unnecessarily complex assertion pattern

Low Severity

The new test manually iterates over call_args_list to collect written lines and then joins them for comparison. This is unnecessarily complex when the same file already has an established simpler pattern at line 107: mock_file.open().writelines.assert_called_once_with(expected.splitlines(keepends=True)). The new test could use mock_path.open().writelines.assert_called_once_with(expected_content.splitlines(keepends=True)) for consistency and simplicity.

Fix in Cursor Fix in Web

@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.57%. Comparing base (2921303) to head (d56bd1a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
- Coverage   97.14%   96.57%   -0.58%     
==========================================
  Files          12       12              
  Lines         701      701              
  Branches       63       63              
==========================================
- Hits          681      677       -4     
- Misses         18       20       +2     
- Partials        2        4       +2     
Flag Coverage Δ
unittests 96.57% <ø> (-0.43%) ⬇️

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.

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.

1 participant