Update jsxBracketSameLine to renamed bracketSameLine#37517
Update jsxBracketSameLine to renamed bracketSameLine#37517waviaei wants to merge 1 commit intoWordPress:trunkfrom
Conversation
|
@waviaei, thank you for the report. We are using a fork of Prettier in WordPress: Line 195 in 9872a70 It's pinned to an older version so it still uses the old flag name. It's about time to make a hard decision and change the existing JavaScript coding styles for WordPress to align with what the original Prettier can support. That's the only way to always stay at the most recent version of Prettier. Related issue: #21872. |
|
I opened #37607 to start making the official proposal to switch to the original Prettier in the most recent 2.5 version. The PR is gigantic and requires input from the WordPress community. It includes also the changes proposed in this PR. |
|
I have an alternative PR that would make the config compatible with both Prettier and the WordPress specific fork at #40479. Thank you for raising this issue. I will close this PR. |
Description
For #37516
How has this been tested?
Screenshots
Types of changes
Renamed
jsxBracketSameLineoption tobracketSameLineas per change in Prittier.Checklist:
*.native.jsfiles for terms that need renaming or removal).