Skip to content

Conversation

@awrichar
Copy link
Contributor

No description provided.

Signed-off-by: Andrew Richardson <andrew.richardson@kaleido.io>
Signed-off-by: Andrew Richardson <andrew.richardson@kaleido.io>
@codecov-commenter
Copy link

codecov-commenter commented Oct 25, 2021

Codecov Report

Merging #287 (489d3fd) into main (818b5ed) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 489d3fd differs from pull request most recent head 680bd8d. Consider uploading reports for the commit 680bd8d to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##              main      #287   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          225       225           
  Lines        12406     12417   +11     
=========================================
+ Hits         12406     12417   +11     
Impacted Files Coverage Δ
internal/apiserver/route_post_token_transfer.go 100.00% <ø> (ø)
pkg/fftypes/tokenaccount.go 100.00% <ø> (ø)
internal/assets/manager.go 100.00% <100.00%> (ø)
internal/assets/token_pool.go 100.00% <100.00%> (ø)
internal/assets/token_pool_created.go 100.00% <100.00%> (ø)
internal/assets/token_transfer.go 100.00% <100.00%> (ø)
internal/database/sqlcommon/tokenaccount_sql.go 100.00% <100.00%> (ø)
internal/database/sqlcommon/tokentransfer_sql.go 100.00% <100.00%> (ø)
internal/events/tokens_transferred.go 100.00% <100.00%> (ø)

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 818b5ed...680bd8d. Read the comment docs.

Copy link
Contributor

@nguyer nguyer left a comment

Choose a reason for hiding this comment

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

Looks good to me

@awrichar awrichar merged commit 605e258 into hyperledger:main Oct 26, 2021
@awrichar awrichar deleted the namespace branch October 26, 2021 17:09
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.

4 participants