Skip to content

Conversation

@jaerod95
Copy link
Contributor

@jaerod95 jaerod95 commented Apr 7, 2025

Moves Legacy CLI into a legacy folder and provides a new entrypoint to the beta CLI

  • adds a --legacy (-l) CLI option that forces the application to run in legacy mode.
  • adds a new warning message when the CLI is run in legacy mode.

Overview

Creates separate entrypoints for the CLI application to support legacy and beta versions. The CLI now runs pointed at the beta API by default in the upcoming 5.0 version. Passing in a new --legacy (-l) flag boots the CLI into a version supporting legacy configurations.

Additionally, this PR moves all legacy implementation code into a lib/legacy directory. All new source code will be written in the lib/src directory to provide full flexibility to rebuild the CLI from the ground up.

Test Plan

  • Ensure running yarn start --legacy boots the application into the legacy version of the CLI. A warning message should log to the output.
  • The legacy application should operate as normal when using the --legacy flag (test pull)
  • Omitting the --legacy flag should boot the application into beta mode. Currently only the --help command is supported.

… to the beta CLI

- adds a `--legacy (-l)` CLI option that forces the application to run in legacy mode.
- adds a new warning message when the CLI is run in legacy mode.
@jaerod95 jaerod95 requested a review from marla-hoggard April 7, 2025 19:09
@jaerod95 jaerod95 requested review from asnewman and removed request for asnewman April 7, 2025 19:09
@jaerod95 jaerod95 changed the base branch from master to v5-beta April 7, 2025 19:10
@marla-hoggard
Copy link
Contributor

@jaerod95 Are there docs somewhere for how to run the local cli? I tried searching in notion but didn't find much and what I did find was really old so not sure if it's still accurate.

I will test this out for my own understanding, but I don't feel comfortable signing off on this PR when I've never used the legacy cli before. I think someone who knows what it's meant to look like/how it works should test this PR to ensure it still works properly.

Co-authored-by: Marla Hoggard <[email protected]>
@jaerod95 jaerod95 merged commit b2db569 into v5-beta Apr 8, 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.

3 participants