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

Unexpected changes to formatted result after bumping to v4 #71

Closed
robwise opened this issue Jun 4, 2017 · 3 comments
Closed

Unexpected changes to formatted result after bumping to v4 #71

robwise opened this issue Jun 4, 2017 · 3 comments

Comments

@robwise
Copy link

robwise commented Jun 4, 2017

  • prettier-eslint-cli version: 4.0.3
  • prettier version: 1.4.2
  • eslint version: 3.19.0

Relevant code/config.

// before
type foo = 'hello world';
type bar = { baz: null };

// after
type foo = "hello world";
type bar = {baz: null};

What you did: I bumped from 3.6.0 to 4.0.3 and ran the formatter. Reverting to 3.6.0 solves all of the below problems.

What happened:

  • Flow annotations' quotations changed to double quotes
  • Flow annotations lost bracket spacing
  • Many instances of going over max line length when it didn't before. These cases aren't the usual instance where prettier formats it in a way that is below line length but then eslint fixes it in a way that is longer, these are cases where prettier should have wrapped itself before it even got to the eslint step.

I can't be sure, but it seems as though the prettier eslint inference isn't working right or something? And eslint --fix is covering it up (except for these issues)?

@kentcdodds
Copy link
Member

I wonder whether this is related to the v6.2.1 release of prettier-eslint which was supposed to be non-breaking, but probably wasn't 🤔

mrm007 added a commit to mrm007/prettier-eslint-cli that referenced this issue Jun 7, 2017
@mrm007
Copy link
Contributor

mrm007 commented Jun 7, 2017

I got you.

@kentcdodds
Copy link
Member

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

No branches or pull requests

3 participants