Skip to content

Wr/smarter suggestion #593

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

Merged
merged 5 commits into from
Jul 19, 2024
Merged

Wr/smarter suggestion #593

merged 5 commits into from
Jul 19, 2024

Conversation

WillieRuemmele
Copy link
Contributor

@W-15921199@

parses error message to find when the user didn't encapsulate strings with spaces correctly
BEFORE:

 ➜  sf org logout --target-org test org my demo org too --all
Error (2): Unexpected arguments: org, org, my, demo, too
See more help with --help

AFTER

 ➜  ../../oss/plugin-auth/bin/run.js org logout --all --target-org test org my demo org too                   
Error (2): Unexpected arguments: org, org, my, demo, too
See more help with --help

Try this:

--target-org "test org my demo org too"

@shetzel
Copy link
Contributor

shetzel commented Jul 19, 2024

QA:
bin/run.js org logout --target-org test org my demo suggests --target-org "test org my demo" ✅ bin/run.js org refresh sandbox --target-org test org --name blahsuggests--target-org "test org"

Trying to break...
bin/run.js org refresh sandbox --target-org test org --name foo bar suggests `--target-org "test org bar"

@shetzel
Copy link
Contributor

shetzel commented Jul 19, 2024

Retesting fixes the usecase that didn't work before. Merging.

@shetzel shetzel merged commit 81cac38 into main Jul 19, 2024
47 checks passed
@shetzel shetzel deleted the wr/smarterSuggestion branch July 19, 2024 21:08
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.

2 participants