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

Closes #2576. Update link to security info on z.cash website. #2649

Merged
merged 1 commit into from Nov 10, 2017

Conversation

Projects
None yet
5 participants
@bitcartel
Contributor

bitcartel commented Oct 3, 2017

No description provided.

@bitcartel bitcartel added this to the 1.0.13 milestone Oct 3, 2017

@bitcartel bitcartel requested a review from daira Oct 3, 2017

@str4d

Concept ACK but needs rewording.

Show outdated Hide outdated README.md Outdated
@str4d

str4d approved these changes Nov 1, 2017

@bitcartel

This comment has been minimized.

Show comment
Hide comment
@bitcartel

bitcartel Nov 6, 2017

Contributor

@daira Quick review? Thanks.

Contributor

bitcartel commented Nov 6, 2017

@daira Quick review? Thanks.

@ebfull

This comment has been minimized.

Show comment
Hide comment
@ebfull

ebfull Nov 9, 2017

Contributor

Can we avoid accidental redundancy/inconsistency between these two pages?

Contributor

ebfull commented Nov 9, 2017

Can we avoid accidental redundancy/inconsistency between these two pages?

@arcalinea

This comment has been minimized.

Show comment
Hide comment
@arcalinea

arcalinea Nov 9, 2017

Contributor

utACK

Contributor

arcalinea commented Nov 9, 2017

utACK

@str4d

This comment has been minimized.

Show comment
Hide comment
@str4d

str4d Nov 9, 2017

Contributor

@ebfull not trivially, as we can't easily substitute anything into the README. If you want to investigate that, please open a new issue to track it.

@zkbot r+

Contributor

str4d commented Nov 9, 2017

@ebfull not trivially, as we can't easily substitute anything into the README. If you want to investigate that, please open a new issue to track it.

@zkbot r+

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Nov 9, 2017

Contributor

📌 Commit 1051242 has been approved by str4d

Contributor

zkbot commented Nov 9, 2017

📌 Commit 1051242 has been approved by str4d

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Nov 9, 2017

Contributor

⌛️ Testing commit 1051242 with merge 064600e...

Contributor

zkbot commented Nov 9, 2017

⌛️ Testing commit 1051242 with merge 064600e...

zkbot added a commit that referenced this pull request Nov 9, 2017

Auto merge of #2649 - bitcartel:2576_security_url, r=str4d
Closes #2576. Update link to security info on z.cash website.
@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Nov 10, 2017

Contributor

☀️ Test successful - pr-merge
Approved by: str4d
Pushing 064600e to master...

Contributor

zkbot commented Nov 10, 2017

☀️ Test successful - pr-merge
Approved by: str4d
Pushing 064600e to master...

@zkbot zkbot merged commit 1051242 into zcash:master Nov 10, 2017

1 check passed

homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment