Polyfills 8.4/8.5: remove overzealous exclusion

I already annotated when I added these attribute related exclusions that they weren't targeted enough and ran the risk of blanket ignoring more than we should. I've now decided to remove these as the excludes are too risky, meaning that end-users may end up missing legitimate PHP cross-version compatibility issues because of it. This highlights that more needs to change in how attributes are detected and flagged in PHPCompatibility itself. Once that improvement is made, these tests should be re-enabled and more targeted excludes should be added. Also see 74, which I have opened as a follow-up to this PR.

Author: jrfnl

PHPCompatibilitySymfonyPolyfillPHP84/ruleset.xml

@@ -24,10 +24,7 @@
  <!--
  Detection for the Deprecated attribute is incomplete in PHPCompatibility 10.0.0-alpha1.
  Handling for this should be added once the detection implementation is known.
- The following exclude is a temporary placeholder, which should be removed as soon as
- possible as it will ignore too much.
  -->
- <exclude name="PHPCompatibility.Attributes.NewAttributes.PHPStormAttributeFound"/>
 
  <exclude name="PHPCompatibility.Classes.NewClasses.reflectionconstantFound"/>
  </rule>

PHPCompatibilitySymfonyPolyfillPHP85/ruleset.xml

@@ -14,10 +14,7 @@
  <!--
  Detection for the DelayedTargetValidation and NoDiscard attributes is incomplete in PHPCompatibility 10.0.0-alpha1.
  This section should be filled out once the detection implementation is known.
- The following exclude is a temporary placeholder, which should be removed as soon as
- possible as it will ignore too much.
  -->
- <exclude name="PHPCompatibility.Attributes.NewAttributes.Found"/>
  </rule>
 
  <!-- Prevent false positives being thrown when run over the code of polyfill-php84 itself. -->

Test/SymfonyPolyfillPHP84Test.php

@@ -24,5 +24,9 @@
 
 $r = new ReflectionConstant(ClassName::CONSTANT_NAME);
 
+/*
+// This test will not help at this time as we can't just ignore the use of the Deprecated attribute
+// with the current attribute handling in PHPCompatibility 10.0.
 #[Deprecated]
 function foo() {}
+*/

Test/SymfonyPolyfillPHP85Test.php

@@ -8,6 +8,10 @@
 array_first($array);
 array_last($array);
 
+/*
+// This test will not help at this time as we can't just ignore the use of the DelayedTargetValidation/NoDiscard attribute
+// with the current attribute handling in PHPCompatibility 10.0.
 #[DelayedTargetValidation]
 #[NoDiscard]
 function dummy() {}
+*/