Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

feat: DI - change verifier interface #3626

Merged

Conversation

mishasizov-SK
Copy link
Contributor

Changed Data Integrity verifier interface.

@mishasizov-SK mishasizov-SK force-pushed the feat_DI_change_verifier branch 4 times, most recently from 6bb4682 to 9b43e91 Compare August 25, 2023 09:52
@codecov
Copy link

codecov bot commented Aug 25, 2023

Codecov Report

Merging #3626 (39b5b15) into main (6911901) will increase coverage by 0.00%.
Report is 1 commits behind head on main.
The diff coverage is 97.67%.

@@           Coverage Diff           @@
##             main    #3626   +/-   ##
=======================================
  Coverage   86.93%   86.93%           
=======================================
  Files         367      367           
  Lines       50024    50029    +5     
=======================================
+ Hits        43487    43495    +8     
+ Misses       4951     4949    -2     
+ Partials     1586     1585    -1     
Files Changed Coverage Δ
component/models/ld/testutil/document_loader.go 82.14% <ø> (ø)
.../models/dataintegrity/suite/ecdsa2019/ecdsa2019.go 95.32% <97.14%> (+3.36%) ⬆️
component/models/dataintegrity/verifier.go 89.88% <100.00%> (-2.71%) ⬇️

"github.com/hyperledger/aries-framework-go/component/models/dataintegrity/models"
"github.com/hyperledger/aries-framework-go/component/models/dataintegrity/suite"
"github.com/hyperledger/aries-framework-go/component/models/ld/processor"
signatureverifier "github.com/hyperledger/aries-framework-go/pkg/doc/signature/verifier"
Copy link
Contributor

@Moopli Moopli Aug 25, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't import afgo pkg into afgo component, use "github.com/hyperledger/aries-framework-go/component/models/signature/verifier"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, done

@fqutishat fqutishat merged commit 5f00b6b into hyperledger-archives:main Aug 28, 2023
13 of 14 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants