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

Use pygments instead of rouge for syntax highlighting #1050

Merged
merged 1 commit into from
May 30, 2016

Conversation

ceedubs
Copy link
Contributor

@ceedubs ceedubs commented May 20, 2016

It appears that GitHub is already using rouge instead of pygments for
the cats site, because it doesn't support pygments. This change is so I
stop getting the following email every time I push the cats website:

You are attempting to use the 'pygments' highlighter, which is currently
unsupported on GitHub Pages. Your site will use 'rouge' for highlighting
instead. To suppress this warning, change the 'highlighter' value to
'rouge' in your '_config.yml' and ensure the 'pygments' key is unset.

It appears that GitHub is already using rouge instead of pygments for
the cats site, because it doesn't support pygments. This change is so I
stop getting the following email every time I push the cats website:

> You are attempting to use the 'pygments' highlighter, which is currently
> unsupported on GitHub Pages. Your site will use 'rouge' for highlighting
> instead. To suppress this warning, change the 'highlighter' value to
> 'rouge' in your '_config.yml' and ensure the 'pygments' key is unset.
@non
Copy link
Contributor

non commented May 30, 2016

Seems reasonable. 👍

@non
Copy link
Contributor

non commented May 30, 2016

I claim this change is a documentation change that only requires 1 sign-off so I'm merging now.

@non non merged commit 3c7dfd0 into typelevel:master May 30, 2016
@non non removed the in progress label May 30, 2016
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.

2 participants