Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch Mathjax JS package source #3755

Open
takluyver opened this issue Jul 12, 2018 · 2 comments
Open

Switch Mathjax JS package source #3755

takluyver opened this issue Jul 12, 2018 · 2 comments

Comments

@takluyver
Copy link
Member

From #3751; cc @minrk @gnestor.

Should we use https://github.com/mathjax/MathJax in bower? This saves Min maintaining the components/Mathjax repository if it works.

Should we move away from Bower entirely? It seems to be no longer the in thing. We tried to move away from it before, and ran into problems, but Kyle mentioned that we were trying to switch away from requirejs at the same time, and that was actually the main issue. I'm not sure which other package manager we'd switch to, though. npm?

@gnestor
Copy link
Contributor

gnestor commented Jul 13, 2018

I have tested using ^2.7.4 vs. components/MathJax#~2.7.4 and the results look identical to me.

As for migrating from bower to npm, it would be nice to switch to npm as that is the current trend in the JS world, but bower works and doesn't require and transpiling (npm packages targeting node need to be transpiled using browerify, webpack, etc.)

@minrk
Copy link
Member

minrk commented Jul 13, 2018

Yeah, switching away from require was the source of the problem, I don't think switching away from bower should be an issue as long as everything is on npm, which is probably is.

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

No branches or pull requests

3 participants