[pp-3] Split up and refactor system_requirements() into OOP hooks

I solved the couple of codesniffing and Stan issues just to see what happens with the tests.

berdir → closed merge request !10557

Started a new merge request, did runtime/update with an internal method that keeps $phase and only removed the install stuff. copied the install requirements from the old MR.

only did enough changes to make phpstan/phpcs happy.

I think this approach makes sense, we are going to have to break this up piece by piece

There are some suspicious failures though.

I think we should do the minimal like you did here then work on organization in follow ups.

nicxvan → changed the visibility of the branch 11.x to hidden.

There is something going on with the file system, all of the failures are related to that, I don't see what is missing in the conversion though.

Ok i got this a lot closer I think nightwatch is failing though.

nicxvan → changed the visibility of the branch 3493718-system-requirements-new to hidden.

It was random.

Appears to have some open threads.

Addressed the comments

believe all feedback has been addressed. Seems like a good conversion least to me.

This will conflict with 📌 Create enums for RequirementSeverity and RequirementPhase Active which is more urgent to get into 11.2.x, so marking this postponed on that one.

Blocker is in. Needs rebase and conflict resolution to use the new enum.

Rebased, this is ready for review again!
Created the follow up: 📌 [pp-1] Deprecate drupal_verify_install_file Active

Still haven't had time for a thorough look at the meat of this MR, but I keep finding nits on the outskirts. 😂 Won't bother NW'ing over these, to encourage real reviews of the real changes, but wanted to open threads for now. Hopefully I can make some time on Tuesday to do a real review of the important parts.

📌 Add value objects to represent the return of hook_requirements Needs work is blocked on this.

I will address @dww's comments now.