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

Headers and list elements that include more than one _ character we… #1231

Merged

Conversation

nathenharvey
Copy link
Contributor

Headers and list elements that include more than one _ character were being rendered incorrectly. (Perhaps it is only when there are an even number but in any case I only found words with either one or two _s.)

The html rendered from markdown would look like:

<h3>be<em>grouped</em>into</h3>

When there is only one _ the html rendered correctly:

<h3>be_directory</h3>

This change escapes the _ when there are multiple.

  • New markdown: ### be\_grouped\_into
  • New html: <h3>be_grouped_into</h3>

resource matchers fixed:

  • be_block_device
  • be_character_device
  • be_grouped_into
  • be_linked_to
  • be_owned_by
  • have_app_pool
  • have_property_value

resource fixed:

  • parse_config_file

Signed-off-by: Nathen Harvey nharvey@chef.io

…re being rendered incorrectly. (Perhaps it is only when there are an even number but in any case I only found words with either one or two `_`s.)

The html rendered from markdown would look like:

`<h3>be<em>grouped</em>into</h3>`

When there is only one `_` the html rendered correctly:

`<h3>be_directory</h3>`

This change escapes the `_` when there are multiple.

* New markdown:  `### be\_grouped\_into`
* New html:      `<h3>be_grouped_into</h3>`

resource matchers fixed:
* be_block_device
* be_character_device
* be_grouped_into
* be_linked_to
* be_owned_by
* have_app_pool
* have_property_value

resource fixed:
* parse_config_file

Signed-off-by: Nathen Harvey <nharvey@chef.io>
@chris-rock
Copy link
Contributor

Thanks @nathenharvey for this fix!

@chris-rock chris-rock merged commit f19c87d into inspec:master Oct 20, 2016
@nathenharvey nathenharvey deleted the nathen/fix_markdown_underscores branch October 20, 2016 19:13
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.

None yet

2 participants