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
Tapable is required by the plugin but listed as peer, forcing users to add it or have npm/yarn complains about unmet peer dependency.
Shouldn't it be a real dependency of the plugin?
The text was updated successfully, but these errors were encountered:
You're probably right, but I'm afraid to change it at this point for fear of breaking builds for people who have it working. Is adding Tapable as a dependency a problem for you?
More an annoyance than a problem, I have no need for tapable but need to include in my dependencies to quiet the builds.
Users who need it should already have it in their dependencies, others shouldn't see any difference.
Tapable is required by the plugin but listed as peer, forcing users to add it or have npm/yarn complains about unmet peer dependency.
Shouldn't it be a real dependency of the plugin?
The text was updated successfully, but these errors were encountered: