Skip to content

Run prettier independently from eslint#4727

Merged
julienw merged 2 commits into
firefox-devtools:mainfrom
julienw:split-prettier-eslint
Sep 1, 2023
Merged

Run prettier independently from eslint#4727
julienw merged 2 commits into
firefox-devtools:mainfrom
julienw:split-prettier-eslint

Conversation

@julienw

@julienw julienw commented Aug 23, 2023

Copy link
Copy Markdown
Contributor

The way we were configuring prettier was useful when prettier wasn't integrated with IDEs much at the time. Nowadays the recommended approach is to run both tools separately.

It's probably better to look at the commits separately.

The only change in a file that doesn't come from prettier is in res/photon/index.html: it looks like that some </div> were unbalanced.

@julienw julienw requested a review from canova August 23, 2023 19:05
@codecov

codecov Bot commented Aug 23, 2023

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (f384166) 88.34% compared to head (9a343bb) 88.34%.

❗ Current head 9a343bb differs from pull request most recent head 0c716c7. Consider uploading reports for the commit 0c716c7 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4727   +/-   ##
=======================================
  Coverage   88.34%   88.34%           
=======================================
  Files         299      299           
  Lines       26726    26726           
  Branches     7213     7213           
=======================================
  Hits        23611    23611           
  Misses       2902     2902           
  Partials      213      213           

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@julienw julienw mentioned this pull request Aug 24, 2023
@julienw julienw force-pushed the split-prettier-eslint branch from 8a8baa5 to 9a343bb Compare August 24, 2023 19:08

@canova canova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@julienw julienw force-pushed the split-prettier-eslint branch from 9a343bb to 70af222 Compare September 1, 2023 15:23
@julienw

julienw commented Sep 1, 2023

Copy link
Copy Markdown
Contributor Author

Thanks!

@julienw julienw enabled auto-merge September 1, 2023 15:24
@julienw julienw force-pushed the split-prettier-eslint branch from 70af222 to 0c716c7 Compare September 1, 2023 15:52
@julienw julienw merged commit f0f4748 into firefox-devtools:main Sep 1, 2023
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.

2 participants