-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat: Mathjax for formules formatting #339
base: master
Are you sure you want to change the base?
feat: Mathjax for formules formatting #339
Conversation
Thanks for the pull request, @vladislavkeblysh! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review.
|
cef5cd1
to
7fdea6a
Compare
78d62f7
to
a341093
Compare
Has problem with CI checker. Tried to use node version 18.19 and npm 9/10. Problems still unsolved |
9f68c68
to
3612fe9
Compare
@vladislavkeblysh I tried pulling this locally and running nvm use
npm ci and I was able to reproduce the error CI is showing. I then ran npm install and then npm ci and the error was no longer there. This makes me think that CI should make it past the current error if you run npm install and check in the updated |
@brian-smith-tcril I did the same thing, but the problem persists. |
@vladislavkeblysh what do you see locally when running rm -rf node_modules
nvm use
npm ci |
We'll probably want product review, here. Mathjax support across the platform is somewhat inconsistent, and we'll likely want to figure out a strategy going forward. |
@jmakowski1123 - is there a product roadmap ticket for this? |
dad60b5
to
21c5cf7
Compare
21c5cf7
to
6d0184e
Compare
Description
Related Pull Requests
Screenshots before:
Screenshots after: