Skip to content

Conversation

@danfimov
Copy link
Contributor

@danfimov danfimov commented Nov 2, 2025

  • Library tomli was listed as a dependency but never imported or used in the codebase;
  • In python 3.11 tomllib released. So if toml parsing will be need - built-in tomllib will be better tool for that since this project uses python 3.13 or higher.

Copilot AI review requested due to automatic review settings November 2, 2025 14:15
Copy link
Contributor

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 removes the tomli dependency from the project's development dependencies. Since the project requires Python 3.13.5+, the standard library's tomllib module (available since Python 3.11) can be used instead for TOML parsing if needed. The removal includes updates to both pyproject.toml and the lock file uv.lock.

  • Removed tomli>=2.2.1 from the dev dependency list in pyproject.toml
  • Updated the lock file to reflect the removal of the tomli package

Reviewed Changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
pyproject.toml Removed tomli from dev dependencies in both [project.dev] and [dependency-groups.dev] sections
uv.lock Removed tomli package definition and references from the lock file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@evalstate evalstate merged commit d373057 into evalstate:main Nov 2, 2025
12 checks passed
@evalstate
Copy link
Owner

Thanks!

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