You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This means that the code from dependencies are bundled with the plugin but same time those dependencies are defined in package.json. This means that the dependencies will be installed to node modules but the used code is already bundled in the plugin! For libraries, I don't think you should use bundle option but let users install the dependencies with their package manager.
Screenshots
If applicable, add screenshots to help explain your problem.
Repro
Version of @smarttools/eslint-plugin-rxjs you are using:
Repository that has a reproduction of the problem:
Note: providing a link to a reproduction of the problem will speed the resolution. It no only helps us see what the problem is and verify that it has been fixed, it helps us create a tests that we can run to verify no changes in the future bring the issue back.
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered:
Describe the bug
This plugin's package size is 16.5 MB according to npm https://www.npmjs.com/package/@smarttools/eslint-plugin-rxjs
It seems that esbuild config uses "bundle: true" setting: https://github.com/DaveMBush/eslint-plugin-rxjs/blob/main/packages/eslint-plugin-rxjs/project.json
This means that the code from dependencies are bundled with the plugin but same time those dependencies are defined in package.json. This means that the dependencies will be installed to node modules but the used code is already bundled in the plugin! For libraries, I don't think you should use bundle option but let users install the dependencies with their package manager.
Screenshots
If applicable, add screenshots to help explain your problem.
Repro
Version of @smarttools/eslint-plugin-rxjs you are using:
Repository that has a reproduction of the problem:
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: