Skip to content

Conversation

@youtux
Copy link
Contributor

@youtux youtux commented Sep 16, 2025

No description provided.

@youtux youtux requested a review from Copilot September 16, 2025 20:18
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 upgrades Poetry from version 2.1.3 to 2.2.0 across the project configuration files. The changes align with Poetry 2.2.0's new features and requirements.

  • Updates Poetry version in pre-commit configuration and GitHub Actions workflow
  • Adds license-files configuration to pyproject.toml
  • Configures dev dependency group as optional and updates installation command

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Adds license-files field and configures dev dependency group as optional
.pre-commit-config.yaml Updates Poetry revision hash from 2.1.3 to 2.2.0
.github/workflows/main.yml Updates Poetry installation version and modifies install command to use --with=dev flag

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

pyproject.toml Outdated
authors = [ { name = "Oleg Pidsadnyi", email= "[email protected]" } ]
maintainers = [ { name = "Alessio Bogon", email = "[email protected]" } ]
license = "MIT"
license-files = [ "LICENSE.md"]
Copy link

Copilot AI Sep 16, 2025

Choose a reason for hiding this comment

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

[nitpick] There's a spacing inconsistency in the license-files array. There should be a space after the opening bracket: [ \"LICENSE.md\" ] to match the formatting style used elsewhere in the file (e.g., line 5 authors array).

Suggested change
license-files = [ "LICENSE.md"]
license-files = [ "LICENSE.md" ]

Copilot uses AI. Check for mistakes.
@youtux youtux force-pushed the bump-poetry branch 2 times, most recently from 33202fe to d544000 Compare September 16, 2025 20:31
@codecov
Copy link

codecov bot commented Sep 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.85%. Comparing base (6635365) to head (b41b4a3).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #253   +/-   ##
=======================================
  Coverage   97.85%   97.85%           
=======================================
  Files          16       16           
  Lines         886      886           
  Branches       54       54           
=======================================
  Hits          867      867           
  Misses          8        8           
  Partials       11       11           

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

@youtux youtux merged commit 059bb0c into master Sep 16, 2025
5 checks passed
@youtux youtux deleted the bump-poetry branch September 16, 2025 20:40
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