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

prettier-ignore-start does not seem to work #225

Closed
plmrry opened this issue May 7, 2021 · 0 comments · Fixed by #226
Closed

prettier-ignore-start does not seem to work #225

plmrry opened this issue May 7, 2021 · 0 comments · Fixed by #226
Labels
enhancement New feature or request

Comments

@plmrry
Copy link

plmrry commented May 7, 2021

Describe the bug
In the HTML of a Svelte file, <!-- prettier-ignore --> works but <!-- prettier-ignore-start --> does not seem to disable Prettier.

To Reproduce

<!-- prettier-ignore-->
<div class="this works this works this works this works this works this works this works this works this works" />

<!-- prettier-ignore-start -->
<div
  class="still gets formatted still gets formatted still gets formatted still gets formatted still gets formatted "
/>
<!-- prettier-ignore-end -->

System (please complete the following information):

  • OS: MacOS
  • IDE: VSCode
  • Plugin/Package: Svelte for VS Code v104.12.0
@dummdidumm dummdidumm transferred this issue from sveltejs/language-tools May 10, 2021
@dummdidumm dummdidumm added the enhancement New feature or request label May 10, 2021
dummdidumm pushed a commit to dummdidumm/prettier-plugin-svelte that referenced this issue May 10, 2021
`<!-- prettier-ignore-start --> ... <!-- prettier-ignore-end -->` is now supported at HTML top level
Closes sveltejs#225
dummdidumm added a commit that referenced this issue May 10, 2021
`<!-- prettier-ignore-start --> ... <!-- prettier-ignore-end -->` is now supported at HTML top level
Closes #225
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants