Skip to content

Conversation

@textbook
Copy link
Contributor

These had previously been introduced for *.mdx files (see #2917) but seem to no longer be needed. As a result, the rendered output includes redundant escapes that would break file matching, e.g.:

module.exports = {
  module: {
    rules: [
      { test: /\\.css$/, use: 'css-loader' },
      { test: /\\.ts$/, use: 'ts-loader' }
    ]
  }
};

See e.g. this Stack Overflow question for resulting confusion.

These had previously introduced for *.mdx files but seem to no longer be
needed.
@vercel
Copy link

vercel bot commented Nov 18, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/webpack-docs/webpack-js-org/qs3bojg3k
✅ Preview: https://webpack-js-org-git-fix-backslash.webpack-docs.vercel.app

@chenxsan
Copy link
Member

Thank you for taking your time, seems like mdx fixed the bug this year mdx-js/mdx#991 and we failed to update those changes.

@chenxsan chenxsan merged commit 18e8e38 into webpack:master Nov 19, 2020
@chenxsan
Copy link
Member

@textbook Thanks for your contribution to webpack documentation.

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.

3 participants