Skip to content
This repository has been archived by the owner on Aug 5, 2020. It is now read-only.

Latest version of mobify.js somewhat minified? #254

Open
zrw354 opened this issue Mar 5, 2014 · 4 comments
Open

Latest version of mobify.js somewhat minified? #254

zrw354 opened this issue Mar 5, 2014 · 4 comments

Comments

@zrw354
Copy link

zrw354 commented Mar 5, 2014

The latest mobify.js on Github is partially minified. Are you guys sure this is a version for human eyes? It seems that you've just decided to not use almond.js, though.

@jansepar
Copy link
Contributor

jansepar commented Mar 5, 2014

@zrw354 I actually never noticed that :). We moved away from Almond.js and went with Browserify. Seems like the loader they insert comes in minified. Adding to my TODO list to check if we can get that snippet unminified. Did a quick googling and couldn't figure it out right away - seems like someone has a similar complaint though: browserify/browserify#623

@zrw354
Copy link
Author

zrw354 commented Mar 5, 2014

Hi jansepar,

Thanks for the quick response.

I forked mobify/mobifyjs last week, with mobify.js being the version based on almond.js. I'd like to catch up with your latest version.

Regards,
zrw354

@jansepar
Copy link
Contributor

jansepar commented Mar 5, 2014

Cool, feel free to rebase what's in the v2.0 branch!

@jansepar jansepar closed this as completed Mar 5, 2014
@jansepar jansepar reopened this Mar 5, 2014
@terinjokes
Copy link

@jansepar you can provide an opts.prelude (and opts.preludePath, for sourcemaps) to browserify's API that accepts a string version of the prelude. The repo (and the package) of browser-pack includes the unminified source.

I do not know if you can provide custom options to grunt-browserify very easily, but I'll leave that to you. ✨

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

3 participants