Skip to content
This repository has been archived by the owner on Oct 29, 2024. It is now read-only.

Update to prettier@2 and ensure it is enforced by eslint #263

Merged
merged 2 commits into from
Apr 2, 2020

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Apr 1, 2020

  • Updates to prettier@ (see blog post for details).
  • Fix issue with prettier linting setup (prettier was not being enforced by eslint prior to these changes).
  • Run yarn lint --fix to fix all the files.

This will be easiest to review by looking at the first commit on its own, as the second commit is eslint --fix . basically.

@rwjblue
Copy link
Member Author

rwjblue commented Apr 1, 2020

Failures are from files that should be excluded from prettier (babel plugin fixture files that are asserting the expected output from babel).

* Updates to prettier@ (see [blog
  post](https://prettier.io/blog/2020/03/21/2.0.0.html) for details).
* Fix issue with prettier linting setup (prettier was not being enforced
  by `eslint` prior to these changes).
* Fix typo in `.eslintignore` (`fixture-options` -> `fixtures-options`)
@rwjblue rwjblue force-pushed the update-prettier branch 2 times, most recently from 3b02606 to 915b15b Compare April 2, 2020 14:27
@rwjblue rwjblue merged commit 29af036 into glimmerjs:master Apr 2, 2020
@rwjblue rwjblue deleted the update-prettier branch April 2, 2020 14:55
@rwjblue rwjblue changed the title Update to prettier@2 Update to prettier@2 and ensure it is enforced by eslint Apr 3, 2020
@rwjblue rwjblue added bug and removed internal labels Apr 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant