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

Add Prettier formatter for JS files #3109

Closed
wants to merge 7 commits into from
Closed

Add Prettier formatter for JS files #3109

wants to merge 7 commits into from

Conversation

montogeek
Copy link
Member

Continues #2247

@netlify
Copy link

netlify bot commented Jun 10, 2019

Preview is ready

Built with commit 8212408

https://deploy-preview-3109--webpackjsorg-netlify.netlify.com

Copy link
Member

@anshumanv anshumanv left a comment

Choose a reason for hiding this comment

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

Have a PR open for this already since days - #3041

@anshumanv
Copy link
Member

lmk if that needs to be closed 😄

{
"singleQuote": true,
"printWidth": 100
}
Copy link
Member

@alexander-akait alexander-akait Jun 27, 2019

Choose a reason for hiding this comment

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

I recommend use this too:

  trailingComma: 'es5',
  arrowParens: 'always',
  • Less changes in review
  • Use parens for consistently output
  • Also i recommend don't use printWidth: 100 with spaces, it can generate unreadable code

Copy link
Contributor

@tryzniak tryzniak 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! Better now than never haha. Also I agree with @evilebottnawi

@montogeek montogeek changed the title Add Prettier formatter Add Prettier formatter for JS files Oct 25, 2019
@montogeek
Copy link
Member Author

@evilebottnawi @anshumanv @EugeneHlushko Ping

@anshumanv I would like to include all your changes, let's do it after merging this one.

@anshumanv
Copy link
Member

WIll update 👍

@jeffin143
Copy link
Contributor

What is the blockage here ?

@chenxsan chenxsan mentioned this pull request Oct 9, 2020
52 tasks
@alexander-akait
Copy link
Member

@montogeek Why prettier is blocked?

@chenxsan I think we need focus on this after webpack@5 upgrading

@chenxsan
Copy link
Member

chenxsan commented Oct 9, 2020

Sure, I can do it after the webpack v5 upgrade if @montogeek doesn't have time to continue it.

@montogeek
Copy link
Member Author

There is no blocker at this moment, I will restart it instead, can do it soon :)

@chenxsan
Copy link
Member

@montogeek Hi, do you still have time doing this? Otherwise I can take care of it.

@chenxsan chenxsan mentioned this pull request Feb 2, 2021
@chenxsan
Copy link
Member

chenxsan commented Feb 5, 2021

Closed via #4518

@chenxsan chenxsan closed this Feb 5, 2021
@chenxsan chenxsan deleted the add-prettierjs branch June 24, 2021 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants