-
Notifications
You must be signed in to change notification settings - Fork 134
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
SIWE Release 2.0.4 #79
Conversation
w4ll3
commented
May 12, 2022
•
edited
Loading
edited
- Adds linter and lint the files. #78
- Unwraps SiweResponse for validate function #84
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PRs this release includes look good and the changes here look good. Ran the tests locally and they passed, but it looks like the test runner fails somehow on this release branch, but not on main
. I think this is worth checking out before merging!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An expected error! We can look at how to display it better when tests run.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to do a rebase merge and not squash all the commits
* Add linter and apply format * Removed useless try catch block * Made promise function not async, renamed assert to fail changed let to const * Fix linter issues with tests * Type provider for checkContractWalletSignature
* Updates package version * Unwraps SiweMessage for legacy function