-
Notifications
You must be signed in to change notification settings - Fork 42
fix(nuxt): remove the module as breaking change #181
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
Conversation
This reverts commit #178 as the Nuxt module should be a breaking change.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
feugy
left a comment
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.
Could you change the version number in packages/web/package.json to 1.6.1 please?
So I won't need a new PR to release this package
This reverts commit #178 as the Nuxt module should be a breaking change.
I will open another PR for 2.0 with the Nuxt module + runtime usage, the only breaking change will be:
But the advantage will be that most people will only have to do: