-
Notifications
You must be signed in to change notification settings - Fork 200
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
✨ IMPROVE: improve math styling #369
Conversation
Thanks for submitting your first pull request! You are awesome! 🤗 |
Doesn't seem to work on either count :( |
What was the sticking point for setting up the dev environment? |
Nothing I couldn't handle, just the sheer the amount of tools that I don't (yet) regularly use to install and learn, compared to the expected amount of changes (I incorrectly expected this to be an easy fix). It was also not completely clear what it'd take—I'd need to install pre-commit to compile the assets, that would in turn install the EBP pre-commit hook, and that will bring in require some minification tools (?). |
Are you going off of the instructions on the contributing page? I think the main thing is just to do:
that should be all you need to do. If you want pre-commit to run when you commit things, then first install pre-commit and do:
and it'll run from then on |
Thanks, this worked, and helped me figure out the issues. |
this looks great - thanks for improving this! |
Closes #345, #272.
I didn't figure out how to set up the dev environment, so the compiled assets aren't updated. I hope CI picks it up.
Some motivation: