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

Fix PostCSS VS Code integration #3454

Merged
merged 1 commit into from
Dec 18, 2017
Merged

Fix PostCSS VS Code integration #3454

merged 1 commit into from
Dec 18, 2017

Conversation

ardevelop
Copy link
Contributor

@ardevelop ardevelop commented Dec 11, 2017

Fixes PostCSS integration for VS Code

Extension (prettier-vscode) is querying prettier for supported languages, and is not getting any information about postcss. As a result engineer sees that is no any formatter for postcss:

image

After exposing information that prettier supports postcss everything works fine.

@duailibe
Copy link
Member

Hi @ardevelop, while I think I understand what's this supposed to fix, could you add a description of what exactly is the problem you are facing and why this fixes it?

Thanks for the PR! :)

@ardevelop
Copy link
Contributor Author

@duailibe thanks for feedback, updated description, PTANL

@duailibe
Copy link
Member

Thanks for updating @ardevelop!

@azz what do you think?

@azz
Copy link
Member

azz commented Dec 14, 2017

What does postcss as a language actually mean?

@azz azz merged commit d410116 into prettier:master Dec 18, 2017
@ikatyang ikatyang added this to the 1.10 milestone Jan 2, 2018
@ardevelop ardevelop deleted the fix-postcss-vscode-integration branch April 22, 2018 10:53
@lock lock bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jul 21, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Jul 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants