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 ] Update code examples. #62

Merged
merged 1 commit into from
May 6, 2024
Merged

Conversation

brianbancroft
Copy link
Contributor

Had some time to go through the verify part of the strategy and ran into the same class of issue this evening and found the same type of missing comma, so here's another PR.

Thanks for taking the time to make this library! I hope to get to use it on my sideproject =)

@dev-xo
Copy link
Owner

dev-xo commented May 6, 2024

Thanks, @brianbancroft. Some of the code examples have been moved around over time, and probably some little typos could have been introduced by myself.

Let me know if the documentation makes sense to you, or any other suggestion!

  • Thanks for the catches again!

@dev-xo dev-xo merged commit 184a8eb into dev-xo:main May 6, 2024
4 checks passed
@dev-xo dev-xo changed the title Add another couple commas in the readme [ Fix ] Update code examples. May 6, 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