-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Introduce k6 cloud upload to replace --upload-only #3906
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.
Nit: maybe you could consider switching the upload the test
(in all places) to upload the test data
or even upload the test scripts and resources.
Perhaps it's just me, but upload the test
sounds incomplete. Shrug: Anyway, for sure, this is a non-blocking thing.
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.
LGTM, but I have some request changes
c2f0c13
to
4bbc72f
Compare
What?
It introduces a new cloud subcommand:
k6 cloud upload
to replace the oldk6 cloud --upload-only
.It also marks the old flag as deprecated, so it is no longer displayed in help, and a deprecation message is displayed when used, as follows:
Flag --upload-only has been deprecated, use "k6 cloud upload" instead
.Why?
As part of the task of revamping the k6 cloud commands.
Checklist
make lint
) and all checks pass.make tests
) and all tests pass.Related PR(s)/Issue(s)
N/A