Skip to content

Clean user input for git clone extension#65906

Merged
joaomoreno merged 1 commit intomicrosoft:masterfrom
valtism:master
Aug 6, 2019
Merged

Clean user input for git clone extension#65906
joaomoreno merged 1 commit intomicrosoft:masterfrom
valtism:master

Conversation

@valtism
Copy link
Contributor

@valtism valtism commented Jan 2, 2019

Strip whitespace and "git clone " from start of string. This will allow
VSCode to intelligently handle copy and paste from command line
instructions being pasted into the clone input box.


I've created this PR because there have been a few times I have just copy-pasted a git clone command into the clone box, and the failure message is not very intuitive as to what went wrong.

This is my first PR, so please let me know if anything is missing here, or if the code is not up to standard or could be harmful. I have tried to keep the scope of the input clean as minimal as possible for this reason.

Strip whitespace and "git clone " from start of string. This will allow
VSCode to intelligently handle copy and paste from command line
instructions being pasted into the clone input box.
@joaomoreno joaomoreno added the git GIT issues label Jan 3, 2019
@joaomoreno joaomoreno added this to the Backlog milestone Jan 3, 2019
@joaomoreno
Copy link
Member

joaomoreno commented Aug 6, 2019

Thanks! 🍻

Sorry for taking too long to get here. Part of this code was already in master since the PR was created, but I decided to merge your changes and use your call to trim() the url. Thanks once again!

Related to #77878

@joaomoreno joaomoreno modified the milestones: Backlog, August 2019 Aug 6, 2019
@joaomoreno joaomoreno merged commit 6208e84 into microsoft:master Aug 6, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Mar 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

git GIT issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants