Skip to content

Clarify php cs fix output

Matthias Larisch requested to merge clarify-php-cs-fix-output into master

What does this MR do?

This MR uses git diff to show problems in code style. To be able to do that, it also introduces changes in the base image to make the git command available. Because the app container works on a copy of the code, git needs to be executed inside.

Also, on local testing there was an error found that the web container is not being built anymore. This is fixed here as well...

Tested manually locally.

How confident are you it won't break things if deployed?

(be honest!)

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
Edited by Nick Sellen

Merge request reports