Skip to content

Upgrade prettier to v3#4730

Merged
julienw merged 3 commits into
firefox-devtools:mainfrom
julienw:upgrade-prettier-v3
Sep 1, 2023
Merged

Upgrade prettier to v3#4730
julienw merged 3 commits into
firefox-devtools:mainfrom
julienw:upgrade-prettier-v3

Conversation

@julienw

@julienw julienw commented Aug 24, 2023

Copy link
Copy Markdown
Contributor

Please look at the 3 last commits only, as the 2 first commits are #4727

@julienw julienw force-pushed the upgrade-prettier-v3 branch from 3d4ce29 to c34e6f2 Compare August 24, 2023 19:14
@julienw julienw requested a review from canova August 24, 2023 19:21
@codecov

codecov Bot commented Aug 24, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage: 28.57% and project coverage change: -0.03% ⚠️

Comparison is base (4bd7d2a) 88.34% compared to head (c34e6f2) 88.32%.

❗ Current head c34e6f2 differs from pull request most recent head 93192c1. Consider uploading reports for the commit 93192c1 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4730      +/-   ##
==========================================
- Coverage   88.34%   88.32%   -0.03%     
==========================================
  Files         299      300       +1     
  Lines       26727    26732       +5     
  Branches     7213     7213              
==========================================
- Hits        23612    23611       -1     
- Misses       2902     2908       +6     
  Partials      213      213              
Files Changed Coverage Δ
src/app-logic/url-handling.js 86.74% <ø> (ø)
src/app-logic/web-channel.js 96.77% <ø> (ø)
src/components/app/AppLocalizationProvider.js 92.18% <ø> (ø)
src/components/app/AppViewRouter.js 80.85% <ø> (ø)
src/components/app/AssemblyCodeFetcher.js 77.14% <ø> (ø)
src/components/app/AssemblyViewToggleButton.js 100.00% <ø> (ø)
...nts/app/CurrentProfileUploadedInformationLoader.js 100.00% <ø> (ø)
src/components/app/DetailsContainer.js 100.00% <ø> (ø)
src/components/app/DragAndDrop.js 98.46% <ø> (ø)
src/components/app/Home.js 71.11% <ø> (ø)
... and 82 more

... and 1 file with indirect coverage changes

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

@julienw julienw changed the title Upgrade prettier v3 Upgrade prettier to v3 Aug 25, 2023
Comment on lines +70 to +72
| Sample index | Sample's stack |
| ------------ | ---------------------------------------------------------------------------------------------- |
| 0 | `JS::RunScript [c++] ➡ onLoad [js] ➡ a [js] ➡ b [js]` |

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For this bad formatting, I filed an issue on the prettier repository: prettier/prettier#15338

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.

Thanks for filing it!

@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 for the upgrade!

Comment thread .prettierrc.js Outdated
Comment thread .prettierrc.js Outdated
Comment on lines +70 to +72
| Sample index | Sample's stack |
| ------------ | ---------------------------------------------------------------------------------------------- |
| 0 | `JS::RunScript [c++] ➡ onLoad [js] ➡ a [js] ➡ b [js]` |

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.

Thanks for filing it!

@julienw julienw force-pushed the upgrade-prettier-v3 branch from c34e6f2 to 93192c1 Compare September 1, 2023 16:51
@julienw

julienw commented Sep 1, 2023

Copy link
Copy Markdown
Contributor Author

Thanks!

@julienw julienw merged commit 3358fd5 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