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

Fixed transaction signing #60

Merged
merged 1 commit into from
Apr 11, 2022
Merged

Conversation

icellan
Copy link
Member

@icellan icellan commented Apr 11, 2022

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #60 (ab786c0) into master (236c9f8) will increase coverage by 0.30%.
The diff coverage is 7.69%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   62.47%   62.78%   +0.30%     
==========================================
  Files          95       95              
  Lines        8690     8695       +5     
==========================================
+ Hits         5429     5459      +30     
+ Misses       2911     2878      -33     
- Partials      350      358       +8     
Flag Coverage Δ
unittests 62.78% <7.69%> (+0.30%) ⬆️

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

Impacted Files Coverage Δ
utils/scripts.go 0.00% <0.00%> (ø)
model_draft_transactions.go 74.18% <25.00%> (+5.24%) ⬆️

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 236c9f8...ab786c0. Read the comment docs.

@mrz1836 mrz1836 added the hot-fix Urgent or important fix/patch label Apr 11, 2022
Copy link
Member

@mrz1836 mrz1836 left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit 0ea0bac into master Apr 11, 2022
@mergify mergify bot deleted the siggi/fixed-transaction-signing branch April 11, 2022 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
hot-fix Urgent or important fix/patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants