Add prettier

[janschoenherr]

This can be run with npx prettier . --check or npx prettier . --write

It will currently only check the ./src folder.

[hansmorb]

Hält sich das denn an in der .editorconfig definierten Codestyle?

[datengraben]

@janschoenherr From my perspective you can add the includes and tests folder also.

• Question: What are you thoughts on configuring it only for php or for all of html/js/php?
  The template files php and js maybe incompatible with getting formatted.

• Question: What @janschoenherr and @hansmorb think if we add in the next step the git-blame ignore list.
  Github Docs

  Because in the next step we are going to commit big changes into the log that are only cosmetic refactorings. They will pollute the git-blame view. A file with the commits that are cosmetic nature, seems to be a good practice. gutenberg Repo

  But that does not have to be part of this PR.

[janschoenherr]

We are using it for JavaScript, JSON, etc. however we did have problems in the past with php template files (html and php in one file). I can't recall specifics though. So we might want to try those too.

@hansmorb We would probably have to adjust the .editorconfig file to match the behavior. Personally, I am not sure you need so many fine grained settings in that file, if you are combining it with prettier.

Regarding the git-blame ignore list. I didn't know about that :-) Seems to make sense! As you said, I would separate the actual prettier run from this PR though.

I updated the PR to include /includes and /tests/php folders.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.32%. Comparing base (6307143) to head (dbf8002).

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #1687   +/-   ##
=========================================
  Coverage     50.32%   50.32%           
  Complexity     2723     2723           
=========================================
  Files            99       99           
  Lines         11317    11317           
=========================================
  Hits           5695     5695           
  Misses         5622     5622           

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

[hansmorb]

Du hast doch auch PHPStorm, entspricht dass, was Prettier aus dem Code machen will dem in der .editorconfig definierten Codestyle? Wenn ja passt das. Wenn nein müssen wir nochmal überlegen, da wir uns eigentlich geeinigt hatten uns an diesen Codestyle zu halten.