Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Wr/delete #199
Wr/delete #199
Changes from 5 commits
2eb3f76
d4004db
ced2483
fe37406
fe5eee2
82e3745
312e6b9
fca05e4
717db9a
de9bd53
a20f8b6
d0bf60a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
What if we just used a private for this and
resolveSuccess
would check that too? Maybe:this.canceled = true; return;
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.
This feels out of place in a results formatting function, and I don't think the comment applies anymore. What if we move this to either the
resolveSuccess
method or therun
method just after callingresolveSuccess()
?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've always wondered why CLI commands use
prompt
instead ofconfirm
(and then have to do the extra casing, etc).source: https://github.com/oclif/cli-ux/blob/fd0ca55f2ae7096ea273592be35e071ee4ddcb0d/src/prompt.ts#L132
ex in use
https://github.com/forcedotcom/source-tracking/blob/e73b17609ae410be4043b423b8488968a7513afe/src/commands/force/source/tracking/reset.ts#L44
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.
Oh, it's probably just a carry-over from TB, I'll swap it for
confirm