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

replace http with https in install.rst #333

Merged
merged 1 commit into from
Dec 3, 2019
Merged

replace http with https in install.rst #333

merged 1 commit into from
Dec 3, 2019

Conversation

fnlctrl
Copy link
Contributor

@fnlctrl fnlctrl commented Nov 29, 2019

Type of change

  • Documentation update

Description

curl and running a bash script from http is way too scary.

I ctrl+c-ed in shock the second I pasted the command into my terminal.

Why was it even http in the first place?

Additional details

Related issues

yacovm
yacovm previously approved these changes Nov 30, 2019
Copy link
Contributor

@yacovm yacovm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix

ale-linux
ale-linux previously approved these changes Dec 1, 2019
@lindluni
Copy link
Contributor

lindluni commented Dec 1, 2019

Please follow the steps here to add your signature to your commit, we require all commits to be signed by their authors: https://github.com/hyperledger/fabric/pull/333/checks

Signed-off-by: Alan <fnlctrl@gmail.com>
@fnlctrl fnlctrl dismissed stale reviews from ale-linux and yacovm via 24d1954 December 3, 2019 02:07
@jyellick jyellick merged commit 9fa8172 into hyperledger:master Dec 3, 2019
@fnlctrl fnlctrl deleted the patch-1 branch December 4, 2019 08:07
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.

6 participants