[Discussion] Add singleAttributePerLine: true to prettier config

[aaronb97]

Prettier added this new parameter singleAttributePerLIne a little bit ago and I'm not sure if I'm a fan. I think would be a win for cleaner diffs and PRs though

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (db716e0) 100.00% compared to head (725610b) 100.00%.

Additional details and impacted files

@@            Coverage Diff            @@
##           develop     #1200   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            2         2           
=========================================
  Hits             2         2           

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

[roblafeve]

Prettier added this new parameter singleAttributePerLIne a little bit ago and I'm not sure if I'm a fan. I think would be a win for cleaner diffs and PRs though

Hmmm... yeah that's interesting. Wonder what it would feel like in an actual project. Definitely would clean up diffs.

[roblafeve]

@aaronb97 what's your current thinking on this?

[aaronb97]

I think it would * probably * be good for new projects... but I think adding it to a project that's been around for a while might mess up the git blame too much

[roblafeve]

I feel like this is something we should try out before enabling it in the template

[aaronb97]

I feel like this is something we should try out before enabling it in the template

crema-components it is

[sonarcloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud