- Issue created by @klausi
- Status changed to Needs review
8 months ago 5:23pm 19 April 2024 - 🇦🇹Austria klausi 🇦🇹 Vienna
pull request: https://github.com/pfrenssen/coder/pull/225
- 🇦🇹Austria klausi 🇦🇹 Vienna
Found a better solution by forking our own function declaration multi-line sniff in https://github.com/pfrenssen/coder/pull/226
- Status changed to Fixed
8 months ago 5:41pm 20 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.
- 🇦🇺Australia dpi Perth, Australia
Why a custom sniff over
SlevomatCodingStandard.Functions.RequireTrailingCommaInDeclaration
? - 🇦🇹Austria klausi 🇦🇹 Vienna
Hm, I probably did not find that sniff. Good to know that it exists, but with the fork of the upstream PHPCS sniff we cover all our requirements for Drupal, so I think it is fine.
- 🇦🇺Australia dpi Perth, Australia
Just a note that if one uses the slevomat version also, multiple errors are reported on a line. One for each rule.