-
Notifications
You must be signed in to change notification settings - Fork 4
Expand file tree
/
Copy pathphpcs.xml
More file actions
38 lines (36 loc) · 1.81 KB
/
Copy pathphpcs.xml
File metadata and controls
38 lines (36 loc) · 1.81 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
<?xml version="1.0"?>
<ruleset name="PSR-Wyz-Sharpspring">
<description>A PSR standard with exceptions for things that I don't want to deal with right now.</description>
<file>.</file>
<exclude-pattern>src/examples/*</exclude-pattern>
<exclude-pattern>vendor/*</exclude-pattern>
<!-- If you want to know: it's just PSR2 with a few tweaks. I think. -->
<rule ref="PSR12">
<!--
I make an effort to keep lines under 120 characters, except:
- function declarations, which I don't split into multiple lines
- exceptions thrown, where I don't split the message into a
variable just to keep the line length OK.
I might change my mind one day but I don't think that improves
readability, and I don't think a line length check is very
necessary in general, as long as we can keep the 'real code'
adhering to a rough standard.
-->
<exclude name="Generic.Files.LineLength"/>
<!--
This should be excluded for ValueObjects (which we have not put in
their own separate directory/namespace yet). We have to use
underscores because non-underscore properties are all 'real' values
in the object.
-->
<exclude name="PSR2.Classes.PropertyDeclaration.Underscore"/>
<!--
"Visibility must be declared on all constants if your project
supports PHP 7.1 or later". - this seems weird to me. Should this
warning not be triggered only if the _minimum_ version is 7.1?
That is apparently not the case right now. I'm excluding this
because I don't want to change my code to suit phpcs.
-->
<exclude name="PSR12.Properties.ConstantVisibility.NotFound"/>
</rule>
</ruleset>