-
Notifications
You must be signed in to change notification settings - Fork 3
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
misc dep bumps #111
misc dep bumps #111
Conversation
Deploying ui-kit with Cloudflare Pages
|
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.
Seems like build is failing now, maybe because of yarn version?
package.json
Outdated
"ava": "^5.3.0", | ||
"@typescript-eslint/eslint-plugin": "^7.8.0", | ||
"@typescript-eslint/parser": "^7.8.0", | ||
"ava": "^6.1.3", |
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.
I'd like to try removing ava altogether since we don't have any ava test here anymore afaik
I think it's the TS bump parsing this differently, or this was bumped
I'll come back to this in another meeting :) |
fixes the error reading cosmjs/encoding/build/utf8.d.ts that it thinks is binary due to utf8 text
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.
Sorry for delay, I was having trouble testing with yarn link
. I'm just going to approve because the ui tutorial points to a stable version anyway. When this publishes, I'll validate, publish a new minor version, and update the ui tutorial to use it.
no ticket
Description
Had free hands during SnT
Security Considerations
Scaling Considerations
Documentation Considerations
Testing Considerations