Skip to content

Use explicit Python version during uv setup#3115

Merged
arkid15r merged 2 commits intovacanza:devfrom
arkid15r:update-uv-setup
Dec 7, 2025
Merged

Use explicit Python version during uv setup#3115
arkid15r merged 2 commits intovacanza:devfrom
arkid15r:update-uv-setup

Conversation

@arkid15r
Copy link
Copy Markdown
Collaborator

@arkid15r arkid15r commented Dec 7, 2025

Proposed change

Use explicit Python version during uv setup

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 Dec 7, 2025

Summary by CodeRabbit

  • Chores
    • Updated development setup configuration to explicitly require Python 3.14 when initializing the project environment.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

The Makefile's setup target was modified to explicitly specify Python 3.14 when creating the virtual environment, changing uv venv --clear to uv venv --clear --python 3.14. No other setup flow changes.

Changes

Cohort / File(s) Summary
Python version specification
Makefile
Added explicit Python 3.14 flag (--python 3.14) to virtual environment creation command in setup target

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

ci

Suggested reviewers

  • PPsyrius
  • KJhellico

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding explicit Python 3.14 version specification to the uv venv setup command.
Description check ✅ Passed The description is related to the changeset, restating the main objective with appropriate context about it being a code quality improvement.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 Dec 7, 2025

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 7, 2025

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff            @@
##               dev     #3115   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          306       306           
  Lines        18083     18083           
  Branches      2292      2292           
=========================================
  Hits         18083     18083           

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fad34c3 and 345db53.

📒 Files selected for processing (1)
  • Makefile (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: PPsyrius
Repo: vacanza/holidays PR: 3062
File: scripts/generate_snapshots.py:69-80
Timestamp: 2025-11-17T05:45:27.759Z
Learning: The vacanza/holidays project requires Python 3.10+ as of November 2025 (specified in pyproject.toml as "requires-python = ">=3.10""). Python 3.9 support has been deprecated.
Learnt from: PPsyrius
Repo: vacanza/holidays PR: 2929
File: scripts/generate_release_notes.py:82-82
Timestamp: 2025-09-15T15:54:52.871Z
Learning: The vacanza/holidays project requires Python 3.9+ and uses PyGithub 2.8.1, so there's no need for backward compatibility with PyGithub 1.x when reviewing authentication code.
⏰ Context from checks skipped due to timeout of 300000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Test build on windows-latest

@arkid15r arkid15r marked this pull request as ready for review December 7, 2025 20:34
@arkid15r arkid15r enabled auto-merge December 7, 2025 20:34
Copy link
Copy Markdown
Collaborator

@KJhellico KJhellico 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 Dec 7, 2025
Merged via the queue into vacanza:dev with commit ff4644c Dec 7, 2025
33 checks passed
@arkid15r arkid15r deleted the update-uv-setup branch December 7, 2025 21:40
@KJhellico KJhellico mentioned this pull request Dec 15, 2025
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