Skip to content
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

Shorter installation command #3600

Merged
merged 2 commits into from
Sep 2, 2019
Merged

Shorter installation command #3600

merged 2 commits into from
Sep 2, 2019

Conversation

star2000
Copy link
Contributor

@star2000 star2000 commented Aug 19, 2019

replace

iex (new-object net.webclient).downloadstring('https://get.scoop.sh')

to

iwr -useb get.scoop.sh | iex

More elegant, isn't it?

@r15ch13
Copy link
Member

r15ch13 commented Aug 20, 2019

Looks nicer but also makes it harder to understand.
I would prefer to add it as a shorter alternative instead of replacing it. 😄
The current command could be changed to full length with proper casing:

Invoke-Expression (New-Object System.Net.WebClient).DownloadString('https://get.scoop.sh')

@star2000
Copy link
Contributor Author

star2000 commented Aug 20, 2019

Looks nicer but also makes it harder to understand.
I would prefer to add it as a shorter alternative instead of replacing it. 😄
The current command could be changed to full length with proper casing:

Invoke-Expression (New-Object System.Net.WebClient).DownloadString('https://get.scoop.sh')

@r15ch13

iwr -useb get.scoop.sh | iex

full length is

Invoke-WebRequest -UseBasicParsing get.scoop.sh | Invoke-Expression

iwr is abbreviation for Invoke-WebRequest.
execute Get-Help Invoke-WebRequest to see other alias.

@star2000
Copy link
Contributor Author

@r15ch13 merge or close

@r15ch13 r15ch13 merged commit 3e55a70 into ScoopInstaller:master Sep 2, 2019
@star2000 star2000 deleted the patch-1 branch September 2, 2019 11:38
@rasa
Copy link
Member

rasa commented Sep 2, 2019

This change isn’t in the develop branch.

@r15ch13
Copy link
Member

r15ch13 commented Sep 2, 2019

It's just the readme. No need to merge it to develop 😁

Rajpratik71 pushed a commit to Technology-Notes/lukesampson-scoop-wiki that referenced this pull request May 27, 2020
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.

3 participants