Skip to content

Comments

update to draft 06#7

Merged
jbowes merged 1 commit intomainfrom
draft-06
Sep 15, 2021
Merged

update to draft 06#7
jbowes merged 1 commit intomainfrom
draft-06

Conversation

@jbowes
Copy link
Owner

@jbowes jbowes commented Sep 11, 2021

TODO:

  • update to new examples
  • support all new specialty components for verification
  • sign with specialty components

@codecov
Copy link

codecov bot commented Sep 11, 2021

Codecov Report

Merging #7 (ddbd508) into main (3b7690b) will increase coverage by 3.46%.
The diff coverage is 100.00%.

❗ Current head ddbd508 differs from pull request most recent head 83ebf6e. Consider uploading reports for the commit 83ebf6e to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   57.98%   61.45%   +3.46%     
==========================================
  Files           5        5              
  Lines         338      358      +20     
==========================================
+ Hits          196      220      +24     
+ Misses        106      104       -2     
+ Partials       36       34       -2     
Impacted Files Coverage Δ
canonicalize.go 71.42% <100.00%> (+3.86%) ⬆️
httpsig.go 53.48% <100.00%> (-0.54%) ⬇️
sign.go 62.02% <100.00%> (+5.26%) ⬆️
verify.go 57.28% <100.00%> (+4.70%) ⬆️

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 3b7690b...83ebf6e. Read the comment docs.

@jbowes jbowes force-pushed the draft-06 branch 8 times, most recently from e8fe3d5 to ddbd508 Compare September 15, 2021 09:29
@jbowes jbowes changed the title WIP: update to draft 6 update to draft 06 Sep 15, 2021
@jbowes jbowes merged commit 94cb60a into main Sep 15, 2021
@jbowes jbowes deleted the draft-06 branch September 15, 2021 09:34
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.

1 participant