Skip to content

fix: remove unused dependency from pyproject.toml#340

Merged
shenxianpeng merged 2 commits intomainfrom
bugfix/remove-unsed-deps
Jan 3, 2026
Merged

fix: remove unused dependency from pyproject.toml#340
shenxianpeng merged 2 commits intomainfrom
bugfix/remove-unsed-deps

Conversation

@shenxianpeng
Copy link
Contributor

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

Summary by CodeRabbit

  • Chores
    • Removed unused external dependency to reduce installation footprint.

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

@shenxianpeng shenxianpeng requested a review from a team as a code owner January 3, 2026 20:25
@shenxianpeng shenxianpeng added the bug Something isn't working label Jan 3, 2026
@netlify
Copy link

netlify bot commented Jan 3, 2026 •
edited
Loading

✅ Deploy Preview for commit-check ready!

Name Link
🔨 Latest commit ebc7bd6
🔍 Latest deploy log https://app.netlify.com/projects/commit-check/deploys/69597b9aeb79fa0008731821
😎 Deploy Preview https://deploy-preview-340--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 3, 2026 •
edited
Loading

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

📝 Walkthrough

Walkthrough

The "typer" package was removed from the project dependencies in pyproject.toml, reducing the dependency list from ["pyyaml", "typer"] to ["pyyaml"] only.

Changes

Cohort / File(s) Summary
Dependency Management
pyproject.toml		 Removed typer from the dependencies array; project now depends solely on pyyaml

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A dependency departs, so light and bright,
Typer bids farewell—the code still runs right!
One less package to manage with care,
Leaner projects float through the air. ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing an unused dependency (typer) from pyproject.toml.
✨ Finishing touches
  • 📝 Generate docstrings

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 3, 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

@shenxianpeng shenxianpeng requested a review from Copilot January 3, 2026 20:27
@codecov
Copy link

codecov bot commented Jan 3, 2026 •
edited
Loading

Codecov Report

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

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #340      +/-   ##
==========================================
+ 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.

Copilot AI reviewed Jan 3, 2026
View reviewed changes
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 unused typer dependency from pyproject.toml. The codebase uses argparse for CLI argument parsing instead of typer, making this dependency unnecessary.

Key Changes:

  • Removed typer from the dependencies list in pyproject.toml

@shenxianpeng shenxianpeng changed the title fix: remove unsed dependencies from pyproject.toml fix: remove unsed dependency from pyproject.toml Jan 3, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 3, 2026 •
edited
Loading

CodSpeed Performance Report

Merging #340 will degrade performance by 40%

Comparing bugfix/remove-unsed-deps (ebc7bd6) with main (b25fc7c)

Summary

❌ 1 regression
✅ 166 untouched
⏩ 85 skipped1

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

Benchmarks breakdown

Benchmark BASE HEAD Efficiency
❌ test_main_with_message_empty_string_no_stdin_with_git 4.7 ms 7.8 ms -40%

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 changed the title fix: remove unsed dependency from pyproject.toml fix: remove unused dependency from pyproject.toml Jan 3, 2026
@shenxianpeng shenxianpeng merged commit 8fa65f4 into main Jan 3, 2026
41 of 42 checks passed
@shenxianpeng shenxianpeng deleted the bugfix/remove-unsed-deps branch January 3, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

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.

1 participant

@shenxianpeng