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

Support Prettier <1.8.0 for getSupportInfo #707

Merged
merged 1 commit into from
Apr 25, 2019

Conversation

JHilker
Copy link
Contributor

@JHilker JHilker commented Jan 31, 2019

Quick followup to #706 (comment)

Connected to #710

This adds a fallback to bundledPrettier.getSupportInfo if prettierInstance.getSupportInfo is undefined. Currently this will still have the babel / babylon parser issue of #705, but this will be necessary once that's fixed upstream.

Copy link
Member

@CiGit CiGit left a comment

Choose a reason for hiding this comment

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

This is a solution which doesn't work immediately. This would need an upgraded prettier.

I'm requesting changes only to prevent myself from merging it right now. @CiGit read that

@JHilker JHilker mentioned this pull request Feb 1, 2019
@CiGit
Copy link
Member

CiGit commented Apr 25, 2019

Seems fixed upstream. Thanks!

@CiGit CiGit merged commit 26fd942 into prettier:master Apr 25, 2019
CiGit pushed a commit that referenced this pull request Apr 25, 2019
* Support prettier 1.17

* Update in reaction to new VS code feature to prompt when multiple formatters are installed

* Support Prettier <1.8.0 for getSupportInfo (#707)

* feat: hide statusBarItem for disabled languages (#733)

* Resolve prettier to bundled if findPkg can't find it (#811)

Fixes #522
Slightly different implementation of #561

* Update .vscodeignore

Add .github folder

* Create bare-bones pull request template (#708)

* Create bare-bones pull request template

This template almost certainly needs fleshing out, but I figured a good place to start was making sure people run tests and making sure they update the Changelog.

* Create separate .github folder for issue/pr templates

* add support for quoteProps

* Support prettier 1.17

* Update in reaction to new VS code feature to prompt when multiple formatters are installed

* add support for quoteProps
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.

None yet

2 participants