-
Notifications
You must be signed in to change notification settings - Fork 254
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
feat(cargo-shuttle): beta flag, remove project list pagination logic #1732
feat(cargo-shuttle): beta flag, remove project list pagination logic #1732
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Curious how this will scale if we start to have more modification from the current CLI. I was thinking of having two different implementation of the API instead (ShuttleApiClient
and ShuttleBetaApiClient
basically). This would also make it clearer what isn't implemented vs the other one.
Decided to go with a boolean property like this since the amount of differences thus far is very small.
|
Okay, let's keep it that way for now then. |
8136998
to
a84e586
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project start, list, status works as expected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
72839f0
into
shuttle-hq:feat/shuttle-ecs-common
…1732) * feat(cargo-shuttle): beta flag * nit: client name * fix: delete beta endpoint
Supersedes #1731