Skip to content

Commit 116ee54

Browse files
committed
fix(psalm): do not report unused issue handler suppressions
Signed-off-by: kyteinsky <kyteinsky@gmail.com> Assisted-by: Github Copilot:claude-opus-5
1 parent 199f34c commit 116ee54

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

psalm.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
resolveFromConfigFile="true"
55
findUnusedCode="false"
66
findUnusedBaselineEntry="false"
7+
findUnusedIssueHandlerSuppression="false"
78
phpVersion="8.1"
89
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
910
xmlns="https://getpsalm.org/schema/config"
@@ -31,6 +32,12 @@
3132
<directory name="vendor" />
3233
</extraFiles>
3334
<issueHandlers>
35+
<!--
36+
Needed for the older supported server versions, whose ISchemaWrapper
37+
docblocks still reference the Doctrine schema classes directly. Newer
38+
versions return OCP\DB\Schema\ITable instead, so these suppressions go
39+
unused there, hence findUnusedIssueHandlerSuppression is turned off.
40+
-->
3441
<UndefinedDocblockClass>
3542
<errorLevel type="suppress">
3643
<referencedClass name="OC\AppFramework\OCS\BaseResponse"/>

0 commit comments

Comments
 (0)