Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: incorrect error message for getFeeForMessage #28529

Merged
merged 1 commit into from Oct 21, 2022

Conversation

jstarry
Copy link
Member

@jstarry jstarry commented Oct 21, 2022

Problem

The error message for getFeeForMessage is wrong

@jstarry jstarry added the automerge Merge this Pull Request automatically once CI passes label Oct 21, 2022
@codecov
Copy link

codecov bot commented Oct 21, 2022

Codecov Report

Merging #28529 (fd8705c) into master (831ed96) will increase coverage by 0.1%.
The diff coverage is 50.0%.

@@            Coverage Diff            @@
##           master   #28529     +/-   ##
=========================================
+ Coverage    77.1%    77.3%   +0.1%     
=========================================
  Files          55       55             
  Lines        2934     2934             
  Branches      408      408             
=========================================
+ Hits         2264     2268      +4     
+ Misses        529      525      -4     
  Partials      141      141             

@mergify mergify bot merged commit 49daa59 into solana-labs:master Oct 21, 2022
@jstarry jstarry deleted the web3/fix-err branch October 21, 2022 05:35
gnapoli23 pushed a commit to gnapoli23/solana that referenced this pull request Dec 16, 2022
nickfrosty pushed a commit to nickfrosty/solana that referenced this pull request Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant