-
Notifications
You must be signed in to change notification settings - Fork 61
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
Use the same dev dependencies as meilisearch-js #1368
Comments
I'm finding it really hard to update things one by one, there's always an incompatibility at some level. It's much less work to just update everything at the same time. So that's what I'll do. @Strift I know you've already made your own PR, but it's easier for me to start fresh, so I'll just continue my work on #1369. |
Thanks for opening this issue. 🙏 I created the 3 sub-issues related to the most important parts. Let's keep this to the minimum for now, and we'll see later for the rest. I understand that everything is so deeply linked that it's a bit hard to update only a single thing (hence why I ended up working on the huge #1363). But maybe it's possible to proceed step by step by doing: |
There's still one issue pending. |
Following up to #1360 (comment)
The text was updated successfully, but these errors were encountered: