Closed (fixed)
Project:
Coder
Version:
8.3.x-dev
Component:
Coder Sniffer
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
13 Apr 2024 at 09:54 UTC
Updated:
4 Jun 2024 at 11:42 UTC
Jump to comment: Most recent
Comments
Comment #2
klausipull request: https://github.com/pfrenssen/coder/pull/225
Comment #3
klausiFound a better solution by forking our own function declaration multi-line sniff in https://github.com/pfrenssen/coder/pull/226
Comment #5
klausiMerged the latter one.
Comment #8
dpiWhy a custom sniff over
SlevomatCodingStandard.Functions.RequireTrailingCommaInDeclaration?Comment #9
klausiHm, 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.
Comment #10
dpiJust a note that if one uses the slevomat version also, multiple errors are reported on a line. One for each rule.