Skip to content

Conversation

mgol
Copy link
Contributor

@mgol mgol commented May 6, 2017

What kind of change does this PR introduce?

A docs bugfix.

Did you add tests for your changes?

N/A

If relevant, did you update the README?

Yes.

Summary

PR #227 made the README not broken completely on the npm package page (https://www.npmjs.com/package/style-loader) but didn't make the code block a real code block. Backticks are only used to delimit code blocks when used on the top level of the Markdown file; once they appear in an HTML block they're treated as regular symbols. The proper fix is to use the <code> tag.

Does this PR introduce a breaking change?

No.

Other information

PR webpack#227 made the README not broken completely on the npm package page (https://www.npmjs.com/package/style-loader) but didn't make the code block a real code block. Backticks are only used to delimit code blocks when used on the top level of the Markdown file; once they appear in an HTML block they're treated as regular symbols. The proper fix is to use the `<code>` tag.
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@mgol Thx

@michael-ciniawsky michael-ciniawsky merged commit 75d67a8 into webpack:master May 6, 2017
@michael-ciniawsky michael-ciniawsky changed the title docs(readme): Fix the <style> code block docs(README): fix the <style> code block May 6, 2017
@mgol mgol deleted the patch-1 branch May 6, 2017 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants