Skip to content

feat: [CG-10557] Codegen update command #353

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

Merged
merged 13 commits into from
Feb 11, 2025

Conversation

clee-codegen
Copy link
Contributor

@clee-codegen clee-codegen commented Feb 7, 2025

Motivation

Add a way for users to update their codegen CLI, list versions, and specify versions to update.

Content

Added a update command. Added ability to list versions and to install a specific version.

Testing

Added unit tests.
Ran update locally.

[clee-cg-10557-codegen-update-command] codegen-sdk/$ uv run codegen update --list
   Built codegen @ file:///home/chris/codegen/codegen-sdk
Uninstalled 2 packages in 17ms
Installed 3 packages in 26ms
1.0
1.19.0
1.20.0 (current)
1.21.0
1.22.0
1.22.1
1.22.2
1.23.0
1.24.0
1.24.1

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

Copy link

codecov bot commented Feb 7, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
27 1 26 0
View the top 1 failed test(s) by shortest run time
::tests.unit.codegen.sdk.core.test_directory
Stack Traces | 0s run time
ImportError while importing test module '.../sdk/core/test_directory.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
.../sdk/core/test_directory.py:8: in <module>
    from codegen.sdk.codebase.codebase_graph import CodebaseGraph
E   ModuleNotFoundError: No module named 'codegen.sdk.codebase.codebase_graph'

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@clee-codegen clee-codegen marked this pull request as ready for review February 8, 2025 00:02
@clee-codegen clee-codegen requested a review from a team as a code owner February 8, 2025 00:02
@jayhack jayhack requested a review from codegen-team as a code owner February 9, 2025 19:39
Copy link
Contributor

@jayhack jayhack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @clee-codegen ! Only feedback I'd give is make sure this is reflected in the docs.

See docs/cli and add an entry in there for the command. Thanks!

@clee-codegen clee-codegen requested a review from jayhack February 10, 2025 21:54
@clee-codegen clee-codegen merged commit ad71d73 into develop Feb 11, 2025
26 checks passed
@clee-codegen clee-codegen deleted the clee-cg-10557-codegen-update-command branch February 11, 2025 19:30
Copy link
Contributor

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants