Skip to content

Commit 131a42b

Browse files
minor #29174 Add required key attribute (greg0ire)
This PR was merged into the 2.8 branch. Discussion ---------- Add required key attribute | Q | A | ------------- | --- | Branch? | 2.8 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a I am getting warnings when running tests with recent phpunit versions (since 7.2.0) : > `- Element 'element': The attribute 'key' is required but missing.` This requirement is far from being new, what is recent is phpunit validating its configuration file against the XSD schema. See sebastianbergmann/phpunit@d4484be This is pedantic (not a bugfix), and might be a bit painful to merge upstream, so if you feel like I should target a more recent instead and let the old branches go on with their lives, please tell me. Commits ------- c0733c2 Add required key attribute
2 parents c887558 + c0733c2 commit 131a42b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

phpunit.xml.dist

+2-2
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@
5656
<array>
5757
<element key="time-sensitive">
5858
<array>
59-
<element><string>Symfony\Component\Console</string></element>
60-
<element><string>Symfony\Component\HttpFoundation</string></element>
59+
<element key="0"><string>Symfony\Component\Console</string></element>
60+
<element key="1"><string>Symfony\Component\HttpFoundation</string></element>
6161
</array>
6262
</element>
6363
</array>

0 commit comments

Comments
 (0)