Skip to content

Update dependencies: pre-commit + make update#3258

Merged
arkid15r merged 1 commit intovacanza:devfrom
arkid15r:ark/update-dependencies
Feb 9, 2026
Merged

Update dependencies: pre-commit + make update#3258
arkid15r merged 1 commit intovacanza:devfrom
arkid15r:ark/update-dependencies

Conversation

@arkid15r
Copy link
Copy Markdown
Collaborator

@arkid15r arkid15r commented Feb 9, 2026

Proposed change

Update dependencies: pre-commit + make update

Type of change

  • New country/market holidays support (thank you!)
  • Supported country/market holidays update (calendar discrepancy fix, localization)
  • Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
  • Dependency update (version deprecation/pin/upgrade)
  • Bugfix (non-breaking change which fixes an issue)
  • Breaking change (a code change causing existing functionality to break)
  • New feature (new holidays functionality in general)

Checklist

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 9, 2026 •
edited
Loading

Summary by CodeRabbit

  • Chores
    • Updated development tooling versions and configurations.

Walkthrough

Updates the pyproject-fmt pre-commit hook version from v2.12.1 to v2.14.1 and restructures multiple pyproject.toml configuration sections into consolidated nested structures while removing the authors field from project metadata.

Changes

Cohort / File(s) Summary
Pre-commit Hook Version
.pre-commit-config.yaml		 Bumped pyproject-fmt from v2.12.1 to v2.14.1.
Project Configuration Structure
pyproject.toml		 Consolidated tool configuration sections: setuptools, pytest, coverage, and mypy now use nested structures. Removed authors field from [project] section.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • PPsyrius
  • KJhellico
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main changes: updating pre-commit hook version and running dependency consolidation via make update.
Description check ✅ Passed The description relates to the changeset by stating the proposed change is to update dependencies and run make update, matching the actual file modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 9, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 9, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (df9bc56) to head (ca9577f).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files 
@@            Coverage Diff            @@
##               dev     #3258   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          307       307           
  Lines        18377     18377           
  Branches      2358      2358           
=========================================
  Hits         18377     18377

☔ 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.

@arkid15r arkid15r marked this pull request as ready for review February 9, 2026 04:43
PPsyrius
PPsyrius approved these changes Feb 9, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@PPsyrius PPsyrius left a comment

Choose a reason for hiding this comment

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

LGTM 🛠️

@arkid15r arkid15r added this pull request to the merge queue Feb 9, 2026
Merged via the queue into vacanza:dev with commit a5d43c9 Feb 9, 2026
32 checks passed
@arkid15r arkid15r deleted the ark/update-dependencies branch February 9, 2026 06:45
@arkid15r arkid15r mentioned this pull request Feb 16, 2026
Merged
@coderabbitai coderabbitai bot mentioned this pull request Mar 17, 2026
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PPsyrius PPsyrius PPsyrius approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

@KJhellico KJhellico Awaiting requested review from KJhellico KJhellico is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arkid15r @PPsyrius