Skip to content

Conversation

avimak
Copy link
Collaborator

@avimak avimak commented Feb 29, 2024

Motivation and Resolution

This saves an extra call on RPC for optionally-known information (like the chainId case).
also fixed speck -> spec typo

Checklist:

  • [v ] Performed a self-review of the code
  • [v ] Rebased to the last commit of the target branch (or merged it into my branch)
  • Linked the issues which this PR resolves
  • Documented the changes in code (API docs will be generated automatically)
  • Updated the tests
  • All tests are passing

this saves an extra call on RPC for optionally-known information (like the `chainId` case).
also fixed speck -> spec typo
@avimak avimak requested review from penovicp and tabaktoni February 29, 2024 16:56
@avimak avimak merged commit 7e09d18 into develop Mar 2, 2024
@avimak avimak deleted the fix/spec-version-options branch March 2, 2024 18:07
Copy link
Contributor

github-actions bot commented Mar 2, 2024

🎉 This PR is included in version 6.1.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@tabaktoni
Copy link
Member

This is ok, this change was introduced into #908 and needed to be merged.

Copy link
Contributor

github-actions bot commented Jul 3, 2024

🎉 This issue has been resolved in version 7.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants