Skip to content
New issue

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

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

Already on GitHub? # to your account

move 'react-syntax-highlighter' to devDependencies #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikaello
Copy link
Contributor

@mikaello mikaello commented Oct 23, 2019

Using command: npm install react-syntax-highlighter@10.2.1 --save-dev

Sorry for the mess in #33, I'll keep this branch clean :-)

(and if you're interested in any of the features I have implemented in my own fork (http://github.com/mikaello/modali), I'll be more than happy to prepare PR-s for those of interest).

Resolving issue #28

Using command: npm install react-syntax-highlighter@10.2.1 --save-dev
@jcrowson
Copy link
Contributor

jcrowson commented Nov 7, 2019

Fantastic, @mikaello ! I'll merge this in now.

@dinoshauer
Copy link

@jcrowson gentle bump :) I guess this would fix the security warning for highlight.js

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

Successfully merging this pull request may close these issues.

3 participants