minor #51395 Psalm: Ignore UnusedClass errors (derrabus)

nicolas-grekas · nicolas-grekas · commit c68fb68bae76 · 2023-08-16T12:11:18.000+02:00
This PR was merged into the 5.4 branch. Discussion ---------- Psalm: Ignore UnusedClass errors | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | N/A | License | MIT | Doc PR | N/A `UnusedClass` errors reported by Psalm are not really helpful for us. Symfony is full of classes that are meant to be used downstream. The only usages we have are inside our tests and fxitures, but we chose to exclude those. Let's ignore `UnusedClass` errors. Commits ------- 12beb97 Psalm: Ignore UnusedClass errors
diff --git a/psalm.xml b/psalm.xml
@@ -36,6 +36,15 @@
                 <referencedClass name="UnitEnum"/>
             </errorLevel>
         </UndefinedDocblockClass>
+        <UnusedClass>
+            <errorLevel type="suppress">
+                <!--
+                    Because we don't analyze our tests or fixtures, we will
+                    get a lot of false positives regarding unused classes.
+                -->
+                <directory name="src/Symfony" />
+            </errorLevel>
+        </UnusedClass>
         <UnusedConstructor>
             <errorLevel type="suppress">
                 <!--
