feat(cli): rename 'ogx letsgo' to 'ogx go' with backward-compatible alias - #5815
Conversation
…lias Rename the top-level 'ogx letsgo' subcommand to 'ogx go' and the nested 'ogx stack letsgo' to 'ogx stack go'. Add a deprecated 'ogx letsgo' alias that emits a FutureWarning directing users to 'ogx go'. Update the persist-config distro directory name from 'letsgo-run' to 'go-run' for consistency. Add test coverage for the new deprecation warning. Signed-off-by: Matthew Farrellee <matt@cs.wisc.edu>
leseb
left a comment
leseb
left a comment
There was a problem hiding this comment.
please update the landing page example to use uvx ogx go instead of uvx --from 'ogx[starter]' ogx stack run starter even though it will only work when we ship 1.0.1.
i'm hesitant, it doesn't cover all cases provided by starter, though maybe it covers all the important cases. @raghotham @cdoern @franciscojavierarceo do you also agree with changing the recommended way of getting started? |
|
This pull request has merge conflicts that must be resolved before it can be merged. @mattf please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork |
|
This pull request has been automatically marked as stale because it has not had activity within 60 days. It will be automatically closed if no further activity occurs within 30 days. |
|
This pull request has merge conflicts that must be resolved before it can be merged. @mattf please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork |
Signed-off-by: Matthew Farrellee <matt@cs.wisc.edu>
3 participants
Rename the top-level 'ogx letsgo' subcommand to 'ogx go' and the nested 'ogx stack letsgo' to 'ogx stack go'. Add a deprecated 'ogx letsgo' alias that emits a FutureWarning directing users to 'ogx go'. Update the persist-config distro directory name from 'letsgo-run' to 'go-run' for consistency. Add test coverage for the new deprecation warning.