Skip to content

Upgrade flowkit to v1.18.0#1594

Merged
chasefleming merged 3 commits into
masterfrom
cf/upgrade-flowkit-v1.18.0
May 15, 2024
Merged

Upgrade flowkit to v1.18.0#1594
chasefleming merged 3 commits into
masterfrom
cf/upgrade-flowkit-v1.18.0

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

@chasefleming chasefleming added the dependencies Pull requests that update a dependency file label May 14, 2024
@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 39.35%. Comparing base (f1d7eb0) to head (b670b8c).

Files Patch % Lines
internal/accounts/create-interactive.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1594   +/-   ##
=======================================
  Coverage   39.35%   39.35%           
=======================================
  Files          51       51           
  Lines        3326     3326           
=======================================
  Hits         1309     1309           
  Misses       1850     1850           
  Partials      167      167           
Flag Coverage Δ
unittests 39.35% <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 should have 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 back for flow config init

@chasefleming chasefleming merged commit 1c07fdb into master May 15, 2024
@chasefleming chasefleming deleted the cf/upgrade-flowkit-v1.18.0 branch May 15, 2024 15:15
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