Skip to content

Conversation

@tpvasconcelos
Copy link
Owner

@tpvasconcelos tpvasconcelos commented Dec 26, 2025

This pull request primarily upgrades the codebase to target Python 3.10, updates the usage of the zip function to enforce stricter argument handling, and cleans up type imports now that Literal and TypeAlias are available directly from the standard library in Python 3.10. Additionally, it updates the CI configuration and adds a changelog entry for the Python version bump.

Python version upgrade and code modernization:

  • Set the target-version in ruff.toml to "py310" to target Python 3.10, and updated the changelog to note this upgrade. [1] [2]
  • Replaced imports of Literal and TypeAlias from typing_extensions with imports from the standard library typing throughout the codebase, and updated related type imports for consistency. [1] [2] [3] [4] [5] [6] [7] [8]

Stricter argument handling in zip usage:

CI/CD and workflow changes:

  • Set a global environment variable DIFF_AGAINST: HEAD for the CI job in .github/workflows/ci.yml instead of repeating it for each test step, simplifying the workflow configuration. [1] [2]

Other improvements:

  • Updated the import of get_args to use the standard library typing in Python 3.10.

📚 Documentation preview 📚: https://ridgeplot--363.org.readthedocs.build/en/363/

Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
@github-actions

This comment has been minimized.

Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Signed-off-by: Tomas Pereira de Vasconcelos <tomasvasconcelos1@gmail.com>
Fix histogram bin midpoints + other doc improvements
@codecov
Copy link

codecov bot commented Dec 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (f15be83) to head (6d8fd2a).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #363   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines          659       657    -2     
  Branches        85        85           
=========================================
- Hits           659       657    -2     
Flag Coverage Δ
combined-src 100.00% <100.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.

@tpvasconcelos tpvasconcelos merged commit d09974c into main Dec 26, 2025
19 checks passed
@tpvasconcelos tpvasconcelos deleted the ruff-py310 branch December 26, 2025 10:35
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