Skip to content

fix: add dependencies in pyproject.toml#352

Merged
shenxianpeng merged 1 commit intomainfrom
bugfix/update-deps
Jan 31, 2026
Merged

fix: add dependencies in pyproject.toml#352
shenxianpeng merged 1 commit intomainfrom
bugfix/update-deps

Conversation

@shenxianpeng
Copy link
Contributor

@shenxianpeng shenxianpeng commented Jan 31, 2026 •
edited by coderabbitai bot
Loading

Added tomli dependency for Python versions < 3.11.

closes #350

Summary by CodeRabbit

  • Chores
    • Updated project dependencies to enhance compatibility with Python versions prior to 3.11.

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

@shenxianpeng
fix: add dependencies in pyproject.toml
01e2243 
Added tomli dependency for Python versions < 3.11.

closes #350
@shenxianpeng shenxianpeng requested a review from a team as a code owner January 31, 2026 00:04
@shenxianpeng shenxianpeng added the bug Something isn't working label Jan 31, 2026
@netlify
Copy link

netlify bot commented Jan 31, 2026 •
edited
Loading

✅ Deploy Preview for commit-check ready!

Name Link
🔨 Latest commit 01e2243
🔍 Latest deploy log https://app.netlify.com/projects/commit-check/deploys/697d4723286d8d00085e39f2
😎 Deploy Preview https://deploy-preview-352--commit-check.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 31, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

A conditional dependency for tomli (version ≥1.1.0) is added to pyproject.toml for Python versions below 3.11, addressing a missing module error when the codebase imports tomli as a fallback for parsing TOML files on older Python versions.

Changes

Cohort / File(s) Summary
Dependency Configuration
pyproject.toml		 Added conditional dependency declaration for tomli as a backport for Python < 3.11, resolving ModuleNotFoundError when the fallback import is attempted in config.py.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A tiny hop, a line so small,
Tomli joins the deps after all!
Python nine and ten rejoice,
Now they have a parsing choice! 📦✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: add dependencies in pyproject.toml' clearly describes the main change: adding tomli as a dependency to resolve a ModuleNotFoundError on Python < 3.11.
Linked Issues check ✅ Passed The pull request adds tomli as a conditional dependency for Python < 3.11, directly addressing issue #350 which reports ModuleNotFoundError for tomli on Python 3.9.
Out of Scope Changes check ✅ Passed All changes are scoped to adding the tomli dependency in pyproject.toml; no unrelated modifications are present.
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
  • Commit unit tests in branch bugfix/update-deps

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

sonarqubecloud bot commented Jan 31, 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

codecov bot commented Jan 31, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.75%. Comparing base (29f0669) to head (01e2243).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #352      +/-   ##
==========================================
+ Coverage   87.31%   87.75%   +0.43%     
==========================================
  Files           8        8              
  Lines         694      694              
==========================================
+ Hits          606      609       +3     
+ Misses         88       85       -3

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

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 31, 2026

CodSpeed Performance Report

Merging this PR will degrade performance by 39.94%

Comparing bugfix/update-deps (01e2243) with main (29f0669)

Summary

❌ 1 regressed benchmark
✅ 166 untouched benchmarks
⏩ 85 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ test_main_with_message_empty_string_no_stdin_with_git 4.7 ms 7.7 ms -39.94%

Footnotes

  1. 85 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

@shenxianpeng shenxianpeng merged commit 2ca2701 into main Jan 31, 2026
35 of 36 checks passed
@shenxianpeng shenxianpeng deleted the bugfix/update-deps branch January 31, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ModuleNotFoundError: No module named 'tomli'

1 participant

@shenxianpeng