-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[ci] update lockfile #10234
[ci] update lockfile #10234
Conversation
|
I tried fixing the errors, but we're block on the same issue as #8762 (comment) again. It's erroring about the I'm not really sure how to go here besides downgrading it for now, but I think we should re-think how we infer the schema types as it has caused many problems. Maybe we should manually type it. |
0e8f6a6
to
d367f73
Compare
d367f73
to
d663030
Compare
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.
Merging with approval from Erika on Discord. I've figured out a workaround to fix the schema types issue, so we could remove the transitive dependency and use the types as usual.
This PR is auto-generated by a nightly GitHub action. It should automatically be merged if tests pass.