Skip to content

Improved linting setup

Nick Sellen requested to merge improved-linting-setup into master

What does this MR do?

A few linting/fixing related things:

  • adds an overlay when developing frontend stuff (will show linting errors in your browser)
  • adds php parallel linter which does basic php syntax linting
  • includes the phpstan command with a reasonable configuration
  • adds a few helper scripts:
    • ./scripts/lint (lint everything! php, js..)
    • ./scripts/fix (fix everything! php, js...)
    • ./scripts/lint-php (lint all php)
    • ./scripts/lint-js (lint all js)

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

Seems ok!

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))

Merge request reports