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

chore(SFINT-3308): Update prettier and ran it #80

Merged
merged 1 commit into from
Jul 8, 2020

Conversation

louis-bompart
Copy link
Collaborator

TL;DR:

  • npm i prettier@latest -E -D installed latest prettier version
  • npm run lint-fix format all the project
  • git add . ; git commit add every change and commit.

I choose deliberately to not do any changes to the prettier config and just follow what's recommended by prettier for the new stuff - so don't add anything in prettierrc, in a perfect world, you should not even have one-.

You can find information about the changes here:
https://prettier.io/blog/2020/03/21/2.0.0.html

Copy link
Collaborator

@lbergeron lbergeron left a comment

Choose a reason for hiding this comment

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

It looks good to me. You should update the readme as Prettier dropped the support of NodeJS < 10.

https://github.com/coveo/search-ui-extensions/blob/master/README.md#setup

@louis-bompart
Copy link
Collaborator Author

It looks good to me. You should update the readme as Prettier dropped the support of NodeJS < 10.

https://github.com/coveo/search-ui-extensions/blob/master/README.md#setup

Agreed, but to keep the 'linting' in its own PR, I will modify it in another PR.

Copy link
Collaborator

@mikegron mikegron left a comment

Choose a reason for hiding this comment

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

:sopretty:

Copy link
Collaborator

@lbergeron lbergeron left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@louis-bompart louis-bompart merged commit d6c403c into master Jul 8, 2020
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

3 participants