Skip to content

Upgrade flowkit#1595

Merged
chasefleming merged 3 commits into
feature/stable-cadencefrom
cf/upgrade-flowkit
May 15, 2024
Merged

Upgrade flowkit#1595
chasefleming merged 3 commits into
feature/stable-cadencefrom
cf/upgrade-flowkit

Conversation

@chasefleming

Copy link
Copy Markdown
Member

For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@codecov-commenter

codecov-commenter commented May 14, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 42.13%. Comparing base (b12cc0a) to head (a83b77f).

Files Patch % Lines
internal/accounts/create-interactive.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                   Coverage Diff                   @@
##           feature/stable-cadence    #1595   +/-   ##
=======================================================
  Coverage                   42.13%   42.13%           
=======================================================
  Files                          59       59           
  Lines                        3956     3956           
=======================================================
  Hits                         1667     1667           
  Misses                       2054     2054           
  Partials                      235      235           
Flag Coverage Δ
unittests 42.13% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread internal/config/init.go
}

state, err := flowkit.Init(readerWriter, sigAlgo, hashAlgo)
state, err := flowkit.Init(readerWriter)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this is a case where we want the emulator account

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good catch. Added

@jribbink jribbink left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

feel free to merge after adding the emulator account to flow config init

@chasefleming chasefleming added the dependencies Pull requests that update a dependency file label May 15, 2024
@chasefleming chasefleming merged commit 300d082 into feature/stable-cadence May 15, 2024
@chasefleming chasefleming deleted the cf/upgrade-flowkit branch May 15, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants