remy / inliner

Node utility to inline images, CSS and JavaScript for a web page - useful for mobile sites
MIT License
1.1k stars 165 forks source link

Make `inlinemin:true` default #200

Open josephrocca opened 5 years ago

josephrocca commented 5 years ago

Per this comment, and the associated issue, it seems like a good idea to default this. If this is rejected, can we add a note to the readme to help others save time trying to debug this? Just spent about an hour or so trying to work out why one of my script files was being skipped. Also didn't realise that the options object names are the same as those accepted by the cli until I dug into the source, so mabe that's worth mentioning in the readme too (happy to do a pull request for this if you want). In any case, thanks for the excellent tool!