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

Default values for proseWrap are incorrect #313

Closed
Connorelsea opened this issue Dec 20, 2017 · 4 comments
Closed

Default values for proseWrap are incorrect #313

Connorelsea opened this issue Dec 20, 2017 · 4 comments
Labels
locked Please open a new issue and fill out the template instead of commenting.

Comments

@Connorelsea
Copy link

Connorelsea commented Dec 20, 2017

The default values for "prettier.proseWrap" are incorrect and cause prettier to fail. To get prettier to work in VSCode, I had to set "prettier.proseWrap" to true, which gives me a warning about it not being an accepted value.

image

The error:

  Option "proseWrap" must be of type:
    boolean
  but instead received:
    string

  Example:
  {
    "proseWrap": true
  }
@CiGit
Copy link
Member

CiGit commented Dec 20, 2017

Boolean values are those in prettier @1.8.x
These new values are valid with prettier @1.9.x which is the version we bundle.
Seems you have prettier @1.8 installed in your project. To use it, you should provide a .prettierrc file with at least one (1) rule in it. This will avoid some troubles with settings changes.

@CiGit
Copy link
Member

CiGit commented Jan 17, 2018

Answered

@CiGit CiGit closed this as completed Jan 17, 2018
@emirotin
Copy link

This causes unexpected errors when the project is using an older version of prettier.
Example: I've provisioned a project using CRA about 2 months ago and it got prettier ^1.8.2. Because of the yarn.lock it was never updated to the newer version. Now all of a sudden the extension has updated at some point and started erroring on me in the project it used to work before.

I understand it a non-trivial issue but probably the extension should have the idea of the minimal version of prettier it supports and
a) warn if the local version is lower
b) offer to use the bundled version in this case?

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot added the locked Please open a new issue and fill out the template instead of commenting. label Apr 13, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked Please open a new issue and fill out the template instead of commenting.
Projects
None yet
Development

No branches or pull requests

3 participants