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

Contribute source.postcss and source.css.postcss scopes #13

Merged
merged 1 commit into from Jul 29, 2022
Merged

Contribute source.postcss and source.css.postcss scopes #13

merged 1 commit into from Jul 29, 2022

Conversation

xymopen
Copy link
Contributor

@xymopen xymopen commented Jun 6, 2022

This PR aims to provide source.postcss and source.css.postcss scopes for other extensions and close #6, #8.

Currently this would only benefit Volar (vuejs/language-tools#191) as Vetur (vuejs/vetur@d96067d) and Svelte (sveltejs/language-tools#1044) have chosen to bundle their own grammars.

@slavanossar
Copy link

Any chance we could get this merged @jonathantneal? Thanks!

@jonathantneal jonathantneal merged commit 5d00317 into csstools:master Jul 29, 2022
@jonathantneal
Copy link
Collaborator

Yes. I will look into getting another release out, as well. Thank you, @xymopen !

@ghaschel
Copy link

Any news about the release with these changes? I am trying to add support to PostCSS in my extensions using your syntax highlighting scope.

@kotx
Copy link

kotx commented Mar 11, 2023

Hi, is it possible to get a release published with these changes? It's been a while.

@slavanossar
Copy link

I had to set up a new computer a couple of weeks ago and was stumped why postcss highlighting was broken in Vue SFCs – it had been so long ago I completely forgot I had manually added the changes from this PR to the extension on my other computers. Only remembered after getting the latest notification on this thread.

Can we pretty please get a new release published @jonathantneal? Thanks!

@slavanossar
Copy link

@jonathantneal can we please get a new release, it's been over a year since it was merged. Thanks!

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.

<style> tag support
5 participants