Skip to content

CLI improvements #1228

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.

Sign up for GitHub

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

Jump to bottom
Merged
jayhack merged 2 commits into from
Aug 20, 2025
Merged

CLI improvements #1228

jayhack merged 2 commits into from
Aug 20, 2025

Conversation

@jayhack
Copy link
Contributor

@jayhack jayhack commented Aug 20, 2025

Motivation

Content

Testing

Please check the following before marking your PR as ready for review

  • I have added tests for my changes
  • I have updated the documentation or added new documentation as needed

@jayhack jayhack requested review from a team and codegen-team as code owners August 20, 2025 20:25
@codecov
Copy link

codecov bot commented Aug 20, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 7.63359% with 121 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/codegen/cli/tui/app.py 7.63% 121 Missing ⚠️

@jayhack jayhack merged commit 771ab52 into develop Aug 20, 2025
14 of 18 checks passed
@jayhack jayhack deleted the jay/cli-improvements-2 branch August 20, 2025 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rushilpatel0 rushilpatel0 rushilpatel0 approved these changes

+1 more reviewer

@codegen-team codegen-team codegen-team approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jayhack @codegen-team @rushilpatel0