- Status changed to Needs review
6 months ago 12:52am 3 August 2024 - Status changed to Needs work
6 months ago 4:37pm 5 August 2024 - 🇺🇸United States smustgrave
See that 2 tests were added so would expect both to fail but testProcessChildDefinitionParentInterfaceMismatch passes the test-only feature.
- Status changed to Needs review
5 months ago 10:52am 10 August 2024 - 🇮🇩Indonesia el7cosmos 🇮🇩 GMT+7
the
::testProcessChildDefinitionParentInterfaceMismatch
method is supposed to pass even without the changes - 🇺🇸United States smustgrave
Then why add it? If it's not testing anything new? Expanding test coverage may be considered out of scope.
The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.