Skip to content

Fix spelling issues#2529

Merged
tmm merged 7 commits intowevm:mainfrom
nnsW3:fix-spelling-issues
Jul 24, 2024
Merged

Fix spelling issues#2529
tmm merged 7 commits intowevm:mainfrom
nnsW3:fix-spelling-issues

Conversation

@nnsW3
Copy link
Copy Markdown
Contributor

@nnsW3 nnsW3 commented Jul 24, 2024

Hello
While looking through your docs I found several spelling issues.
Br, Elias,.


PR-Codex overview

This PR updates comments in DeploylessUniversalSigValidator.sol to clarify the order of operations for signature validation based on EIP standards.

Detailed summary

  • Updated a comment to correct a typo from "striclty" to "strictly"
  • Clarified the defined order of operations for signature validation based on EIP-6492

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jul 24, 2024

⚠️ No Changeset found

Latest commit: e2e7e89

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Jul 24, 2024

@nnsW3 is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread site/pages/experimental/eip5792/writeContracts.mdx
Comment thread site/pages/docs/actions/public/watchEvent.md
Comment thread site/pages/docs/actions/public/verifyMessage.md
@nnsW3
Copy link
Copy Markdown
Contributor Author

nnsW3 commented Jul 24, 2024

done

@tmm tmm merged commit f7feeac into wevm:main Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants