Skip to content

Extract the crypto module to a new repository #1803

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

Merged
merged 2 commits into from
Oct 7, 2019
Merged

Extract the crypto module to a new repository #1803

merged 2 commits into from
Oct 7, 2019

Conversation

sgkim126
Copy link
Contributor

@sgkim126 sgkim126 commented Oct 4, 2019

The module is moved to https://github.com/CodeChain-io/rust-codechain-crypto.

This PR closes #1500.

@sgkim126 sgkim126 changed the title Extract the crypto module Extract the crypto module to a new repository Oct 4, 2019
@majecty
Copy link
Contributor

majecty commented Oct 4, 2019

@sgkim126
Should we use the git URL to find the crypto library?
What do you think about uploading the Crypto package in the Cargo registry?

@sgkim126
Copy link
Contributor Author

sgkim126 commented Oct 4, 2019

@majecty I think registering the library in crates.io will merely increase the management cost with little benefit. Others except for CodeChain will rarely use the library since we customized it for CodeChain.

@sgkim126 sgkim126 requested a review from majecty October 4, 2019 02:45
@kseo
Copy link
Member

kseo commented Oct 4, 2019

I agree with @sgkim126.

@sgkim126 sgkim126 merged commit 202bbfc into CodeChain-io:master Oct 7, 2019
@sgkim126 sgkim126 deleted the crypto branch October 7, 2019 04:42
foriequal0 pushed a commit to foriequal0/codechain that referenced this pull request Nov 8, 2019
Extract the crypto module to a new repository
foriequal0 pushed a commit that referenced this pull request Nov 11, 2019
Extract the crypto module to a new repository
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extract the crypto module to a new package
3 participants