Skip to content

Revert "Switch parse-srcset to srcset (#7381)"#8359

Merged
fisker merged 3 commits intoprettier:masterfrom
fisker:revert-7381
May 21, 2020
Merged

Revert "Switch parse-srcset to srcset (#7381)"#8359
fisker merged 3 commits intoprettier:masterfrom
fisker:revert-7381

Conversation

@fisker
Copy link
Copy Markdown
Member

@fisker fisker commented May 20, 2020

Revert #7381, Fixes #8150

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory)
  • (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/pr-XXXX.md file following changelog_unreleased/TEMPLATE.md.
  • I’ve read the contributing guidelines.

Try the playground for this PR

"minimist": "1.2.5",
"n-readlines": "1.0.0",
"outdent": "0.7.1",
"parse-srcset": "ikatyang/parse-srcset#54eb9c1cb21db5c62b4d0e275d7249516df6f0ee",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about countries where github banned? They exist 😄

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

To be honestly we should fix it in the future

Copy link
Copy Markdown
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Anyway let's merge, I have rewritten algorithm in html-loader (based on spec) https://github.com/webpack-contrib/html-loader/blob/master/src/plugins/source-plugin.js#L41, maybe it is time to move it to own package

@fisker fisker changed the title Revert 7381 Revert #7381 May 21, 2020
@fisker fisker merged commit 6d49e3b into prettier:master May 21, 2020
@fisker fisker deleted the revert-7381 branch May 21, 2020 04:25
@fisker fisker changed the title Revert #7381 Revert "Switch parse-srcset to srcset (#7381)" Aug 12, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTML: URLs with commas are broken in srcset

3 participants