-
Notifications
You must be signed in to change notification settings - Fork 82
Should Microdiff on NPM be minified? #8
-
In #5, @simone-sanfratello brought up minifying the code in Microdiff's NPM package. However, most people targeting the web already minify everything themselves, and many people who use Node like their dependencies being readable.
This discussion is for seeing what the community's opinions on this are. If you have an opinion on this, please react with 👍 or 👎 and add any other thoughts in the replies.
Beta Was this translation helpful? Give feedback.
All reactions
-
👍 3 -
👎 9
Replies: 2 comments 1 reply
-
Just to be clear, thumbs down = should not be minified?
Any package installed from NPM should absolutely not be minified. Anyone who is interested in delivering a small app is already using a minifier for the rest of their code. Being able to read a dependency's code (preferably with comments intact) is very valuable, and it also gives cleaner error traces. Minified versions of the package can be included in the NPM distribution for those that want them, but it should not be what is used as the entry point for imports.
Beta Was this translation helpful? Give feedback.
All reactions
-
Yeah, 👎 is saying it should not be minified, as Microdiff currently is
Beta Was this translation helpful? Give feedback.
All reactions
-
Due to the majority that says it should not be minified, I will keep it as it is currently.
Beta Was this translation helpful? Give feedback.