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

Multisig support #10

Merged
merged 16 commits into from
May 3, 2021
Merged

Multisig support #10

merged 16 commits into from
May 3, 2021

Conversation

xandkar
Copy link
Contributor

@xandkar xandkar commented Apr 4, 2021

No description provided.

@codecov
Copy link

codecov bot commented Apr 4, 2021

Codecov Report

Merging #10 (337c320) into master (35ed925) will decrease coverage by 2.44%.
The diff coverage is 93.13%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   97.38%   94.94%   -2.45%     
==========================================
  Files           1        1              
  Lines         153      356     +203     
==========================================
+ Hits          149      338     +189     
- Misses          4       18      +14     
Impacted Files Coverage Δ
src/libp2p_crypto.erl 94.94% <93.13%> (-2.45%) ⬇️

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 35ed925...337c320. Read the comment docs.

@xandkar xandkar force-pushed the xandkar/multisig branch 4 times, most recently from 674b233 to 7b134a4 Compare April 6, 2021 04:41
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
@xandkar xandkar force-pushed the xandkar/multisig branch 4 times, most recently from c3e2d66 to 23de3c5 Compare April 7, 2021 18:37
@xandkar xandkar marked this pull request as ready for review April 7, 2021 18:38
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
@xandkar xandkar force-pushed the xandkar/multisig branch 15 times, most recently from 9f8cfa4 to 043da5b Compare April 8, 2021 20:24
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
@xandkar xandkar force-pushed the xandkar/multisig branch 2 times, most recently from 6627b4d to 1132f0c Compare April 19, 2021 05:16
@Vagabond
Copy link
Member

I think the last thing here is to add deterministic sorting of the keys, as discussed on the call yesterday to help reduce the scope for hash collisions.

.github/workflows/ci.yml Show resolved Hide resolved
src/libp2p_crypto.erl Show resolved Hide resolved
src/libp2p_crypto.erl Show resolved Hide resolved
src/libp2p_crypto.erl Outdated Show resolved Hide resolved
@Vagabond Vagabond merged commit 00394ae into helium:master May 3, 2021
@xandkar xandkar deleted the xandkar/multisig branch May 3, 2021 18:27
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

5 participants