Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

fix: update to post-v0.9 Transaction wire format #22

Merged
merged 3 commits into from
Oct 10, 2018
Merged

Conversation

mvines
Copy link
Contributor

@mvines mvines commented Oct 10, 2018

@mvines
Copy link
Contributor Author

mvines commented Oct 10, 2018

Excellent. This PR failed because the fix for solana-labs/solana#1464 has not been published to the edge docker image yet.

@codecov-io
Copy link

Codecov Report

Merging #22 into dev will increase coverage by 0.34%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #22      +/-   ##
==========================================
+ Coverage   92.79%   93.14%   +0.34%     
==========================================
  Files          11       11              
  Lines         347      350       +3     
  Branches       40       39       -1     
==========================================
+ Hits          322      326       +4     
+ Misses         25       24       -1
Impacted Files Coverage Δ
src/transaction.js 92.3% <100%> (+5.35%) ⬆️

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 88e9575...476108f. Read the comment docs.

@mvines mvines merged commit 3b8fa0a into solana-labs:dev Oct 10, 2018
@mvines mvines deleted the dev branch October 10, 2018 20:25
@mvines
Copy link
Contributor Author

mvines commented Oct 13, 2018

🎉 This PR is included in version 0.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2023

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants