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

Add additional crypto methods to PublicKey. #66

Merged
merged 1 commit into from
May 28, 2018

Conversation

jhtitor
Copy link
Contributor

@jhtitor jhtitor commented Apr 25, 2018

Note: tweak_add will NOT work without secp256k1. We shall add pure-python implementation for such cases.

Note: `tweak_add` will NOT work without secp256k1. We shall
add pure-python implementation for such cases.
@codecov-io
Copy link

Codecov Report

Merging #66 into develop will decrease coverage by 2.13%.
The diff coverage is 10.9%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #66      +/-   ##
===========================================
- Coverage    58.41%   56.28%   -2.14%     
===========================================
  Files           19       19              
  Lines         1171     1226      +55     
===========================================
+ Hits           684      690       +6     
- Misses         487      536      +49
Impacted Files Coverage Δ
graphenebase/account.py 62.56% <28.57%> (-1.27%) ⬇️
graphenebase/ecdsa.py 46.53% <8.33%> (-9.31%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f3a6bd...3ee502c. Read the comment docs.

@xeroc xeroc merged commit f3ac669 into xeroc:develop May 28, 2018
xeroc added a commit that referenced this pull request Jul 11, 2018
Release 0.6.3

28c5796 (HEAD -> master) Merge branch 'release/0.6.3'
1a03fb0 (release/0.6.3) Updated docs
463cf18 Cleanup and linting
13966bf version bump
0b5ce1b (origin/develop, develop) Allow num_retries to be smaller than zero
e9af291 allow to obtain chain_parameters
5ef1830 properly use register_apis and api_id
a176d61 Fix legacy
8033e92 entirely new structure for API calls
f3ac669 Merge pull request #66 from jhtitor/pubkey_tweak
78ae5f9 version bump
b54a128 Allow more flexibility when defining the chain
3ee502c Add additional crypto methods to PublicKey.
8f3a6bd Improve retrying of other nodes
eeef51e Add additional crypto methods to PrivateKey. (#64)
@xeroc
Copy link
Owner

xeroc commented Aug 8, 2018

@jhtitor Is it possible that you provide unit tests for those?

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.

None yet

3 participants