Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate from Typer to Cyclopts #79

Merged
merged 25 commits into from
Feb 12, 2025
Merged

Migrate from Typer to Cyclopts #79

merged 25 commits into from
Feb 12, 2025

Conversation

jayqi
Copy link
Owner

@jayqi jayqi commented Jan 16, 2025

  • Migrate from Typer to Cyclopts as the CLI framework
    • Add support for configuration files via Cyclopts
  • Refactor formatters to be callables

Copy link
Contributor

github-actions bot commented Jan 16, 2025

@github-actions github-actions bot temporarily deployed to pull request January 16, 2025 17:17 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 25, 2025 23:05 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 25, 2025 23:07 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 31, 2025 02:03 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 31, 2025 02:16 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 31, 2025 02:17 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 31, 2025 02:21 Inactive
@github-actions github-actions bot temporarily deployed to pull request January 31, 2025 02:23 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 8, 2025 17:16 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2025

Codecov Report

Attention: Patch coverage is 82.54545% with 48 lines in your changes missing coverage. Please review.

Project coverage is 91.7%. Comparing base (1df6231) to head (fb6b4d8).

Files with missing lines Patch % Lines
reprexlite/cli.py 57.3% 38 Missing ⚠️
reprexlite/formatting.py 93.9% 7 Missing ⚠️
reprexlite/config.py 93.3% 2 Missing ⚠️
reprexlite/ipython.py 93.3% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (1df6231) and HEAD (fb6b4d8). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (1df6231) HEAD (fb6b4d8)
6 5
Additional details and impacted files
@@           Coverage Diff           @@
##            main     #79     +/-   ##
=======================================
- Coverage   96.9%   91.7%   -5.3%     
=======================================
  Files         11      11             
  Lines        693     747     +54     
=======================================
+ Hits         672     685     +13     
- Misses        21      62     +41     
Files with missing lines Coverage Δ
reprexlite/__main__.py 100.0% <100.0%> (+100.0%) ⬆️
reprexlite/exceptions.py 100.0% <100.0%> (ø)
reprexlite/reprexes.py 96.8% <100.0%> (+<0.1%) ⬆️
reprexlite/session_info.py 100.0% <100.0%> (+2.7%) ⬆️
reprexlite/version.py 100.0% <100.0%> (+20.0%) ⬆️
reprexlite/ipython.py 94.2% <93.3%> (ø)
reprexlite/config.py 96.7% <93.3%> (-3.3%) ⬇️
reprexlite/formatting.py 94.8% <93.9%> (-5.2%) ⬇️
reprexlite/cli.py 61.3% <57.3%> (-28.7%) ⬇️

@github-actions github-actions bot temporarily deployed to pull request February 8, 2025 17:24 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 12, 2025 03:01 Inactive
@jayqi jayqi mentioned this pull request Feb 12, 2025
@jayqi jayqi marked this pull request as ready for review February 12, 2025 05:42
@jayqi jayqi merged commit ba286fd into main Feb 12, 2025
19 checks passed
@jayqi jayqi deleted the cyclopts branch February 12, 2025 05:42
This was referenced Feb 12, 2025
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