Whitelist Pygments options #2642

Merged
merged 3 commits into from Jul 29, 2014

Conversation

Projects
None yet
4 participants
@parkr
Member

parkr commented Jul 29, 2014

Whitelists the following:

  • :startinline
  • :hl_linenos
  • :linenos
  • :encoding (which we always override as utf-8 anyway)
  • :cssclass

/cc @mastahyeti @ptoomey3

@parkr parkr merged commit 4df274f into master Jul 29, 2014

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details

@parkr parkr deleted the pygments-opts-sanitize branch Jul 29, 2014

parkr added a commit that referenced this pull request Jul 29, 2014

@albertogg

This comment has been minimized.

Show comment
Hide comment
@albertogg

albertogg Jul 29, 2014

Member

OMG! this is so cool! but how are they meant to be used? through {% highlight %}?

Member

albertogg commented Jul 29, 2014

OMG! this is so cool! but how are they meant to be used? through {% highlight %}?

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Jul 29, 2014

Member

@albertogg The SYNTAX const is still a bit messed up, but the regexp in scan seems to work well for fetching arbitrary params. The params listed above will be available on GHP if/when 2.1 support is added.

Member

parkr commented Jul 29, 2014

@albertogg The SYNTAX const is still a bit messed up, but the regexp in scan seems to work well for fetching arbitrary params. The params listed above will be available on GHP if/when 2.1 support is added.

@albertogg

This comment has been minimized.

Show comment
Hide comment
@albertogg

albertogg Jul 29, 2014

Member

ahh ok, 👍 thanks @parkr!

Member

albertogg commented Jul 29, 2014

ahh ok, 👍 thanks @parkr!

@mastahyeti

This comment has been minimized.

Show comment
Hide comment
@mastahyeti

mastahyeti Jul 30, 2014

Contributor

🆒

Contributor

mastahyeti commented Jul 30, 2014

🆒

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.