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

Pin Mathjax version to 2.7.6 #451

Closed

Conversation

matthewborden
Copy link

Mathjax just updated to version 3.

@dpvc
Copy link
Member

dpvc commented Sep 5, 2019

Thanks for your interest in MathJax and mathjax-node; we appreciate the support. There are some problems with your pull request, however.

First, it is based on a branch that is over two years old (July 2017). Small changes like this should be based on a recent branch. Also, it should be branched from and targeted to the develop branch, not the master branch.

Second, this branch includes commits for additional functionality (the macro reset functionality), which makes this PR effectively the same as PR #348. Your PR should include only the single fix that you have indicated in the title.

Third, the issue with the MathJax version was resolved in commit 3418e25, which set the version to ^2.7.2, so it will get the highest 2.x version, but not 3.0. This was made in Dec 2017, so that is nearly two years ago.

Finally, there are merge conflicts with the branch (see for example the package.json file, where there are conflicts due to the Dec 2017 change and later ones).

For these reasons, this PR will be rejected. The issue you are indicating in your title is already taken care of, so there is no need for a new PR for this.

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

Successfully merging this pull request may close these issues.

3 participants