Messages should have role=status instead of role=contentinfo

Rerolled for 10.1. Applies cleanly.

#46 it's not working

Tests might fail and they will require extra work

Back to NR for the new patches.

The last submitted patch, 48: 2942404-48.patch, failed testing. View results →

The last submitted patch, 49: 2942404-49.patch, failed testing. View results →

Solving failed test cases and rerolling for 11.x

Caused massive test failures.

Was previously tagged for issue summary, has that been completed.

Rerolling to 11.x from #49.

Prashant.c → made their first commit to this issue’s fork.

Submitting Patch for 11.x and raising the MR as well.

Thank you.

Large number of failures.

Also was previously tagged for issue summary update.

@Prashant.c there's no interdiff added what did you do different from #55

I opened MR 6112 as patches don't seem to be testing properly. Most of the 1800+ test failures in #55 and #58 do not seem to have anything to do with the code changes. Among other things there are "missing composer.lock" messages, and that file is not touched in either patch. The MR I created is failing ~3 tests, so those will need to be addressed, but that should be straightforward compared to the unexpectedly huge fail count that suddenly appeared (and not at all the fault of @Prashant.c or @Hardik_Patel_12)

@bnjmnm, I have observed similar challenges in several other issues. When using the patch method, it leads to significant unrelated failures for the composer and other components, similar to the problems described in issues #55 and #58. However, the Merge Request (MR) method is functioning properly.

Bug in core's spellcheck step is blocking tests. 🐛 Spell-checking job fails with "couldn't find remote ref refs/heads/11.x" when 11.x branch does not exist in fork Active