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

Website: Add MathJax #5596

Merged
merged 2 commits into from
Oct 5, 2022
Merged

Website: Add MathJax #5596

merged 2 commits into from
Oct 5, 2022

Conversation

Pandapip1
Copy link
Member

Allows using LaTeX in EIPs (yay!)

@Pandapip1 Pandapip1 requested a review from eth-bot as a code owner September 5, 2022 11:59
@Pandapip1 Pandapip1 changed the title Configuration: Add MathJax Support Configuration: Add MathJax Sep 5, 2022
@eth-bot
Copy link
Collaborator

eth-bot commented Sep 5, 2022

A critical exception has occurred:
Message: pr 5596 is already merged; quitting
(cc @alita-moore, @mryalamanchi)

_layouts/eip.html Outdated Show resolved Hide resolved
@Pandapip1 Pandapip1 changed the title Configuration: Add MathJax Wbsite: Add MathJax Sep 21, 2022
@Pandapip1 Pandapip1 changed the title Wbsite: Add MathJax Website: Add MathJax Sep 21, 2022
Copy link
Contributor

@SamWilsn SamWilsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed in EIPIP 66 (ethereum-cat-herders/EIPIP#181), this is pretty low risk and if we don't like it, we can remove it.

@eth-bot eth-bot enabled auto-merge (squash) October 5, 2022 14:11
@eth-bot eth-bot merged commit 6e8f083 into master Oct 5, 2022
@eth-bot eth-bot deleted the Pandapip1-mathjax branch October 5, 2022 14:11
nachomazzara pushed a commit to nachomazzara/EIPs that referenced this pull request Jan 13, 2023
* Configuration: Add MathJax Support

* Add integrity and crossorigin attrs
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.

4 participants