-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[auto/nodejs] Test remote operations #11292
Conversation
Changelog[uncommitted] (2022-11-09) |
be8522f
to
3eb3b08
Compare
if (args.commitHash && args.branch) { | ||
throw new Error("commitHash and branch cannot both be specified."); | ||
if (args.branch && args.commitHash) { | ||
throw new Error("branch and commitHash cannot both be specified."); | ||
} | ||
if (!args.commitHash && !args.branch) { | ||
throw new Error("at least commitHash or branch are required."); | ||
if (!args.branch && !args.commitHash) { | ||
throw new Error("either branch or commitHash is required."); |
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.
Minor change to reorder the check to match the error message (changed to be consistent with CLI and other languages).
3eb3b08
to
e0179b5
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.
I aspire to write code this clean and thorough! LGTM! One nit about adding a comment.
Also cleans up some error messages to be consistent with the CLI and other languages.
e0179b5
to
31bf640
Compare
bors merge |
11292: [auto/nodejs] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - #11290 - #11291 - #11293 - #11294 11293: [auto/python] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - #11290 - #11291 - #11292 - #11294 Co-authored-by: Justin Van Patten <jvp@justinvp.com>
Build failed (retrying...): |
Build succeeded: |
11290: [cli] Test remote operations r=justinvp a=justinvp Also some changes to validate flags earlier (before getting the backend). Related: - #11291 - #11292 - #11293 - #11294 11293: [auto/python] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - #11290 - #11291 - #11292 - #11294 Co-authored-by: Justin Van Patten <jvp@justinvp.com>
11291: [auto/go] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - #11290 - #11292 - #11293 - #11294 11294: [auto/dotnet] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - #11290 - #11291 - #11292 - #11293 Co-authored-by: Justin Van Patten <jvp@justinvp.com>
11291: [auto/go] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - pulumi/pulumi#11290 - pulumi/pulumi#11292 - pulumi/pulumi#11293 - pulumi/pulumi#11294 11294: [auto/dotnet] Test remote operations r=justinvp a=justinvp Also cleans up some error messages to be consistent with the CLI and other languages. Related: - pulumi/pulumi#11290 - pulumi/pulumi#11291 - pulumi/pulumi#11292 - pulumi/pulumi#11293 Co-authored-by: Justin Van Patten <jvp@justinvp.com>
Also cleans up some error messages to be consistent with the CLI and other languages.
Related: