Finalize \Drupal\automatic_updates\Development\Converter script to update core MR

Added to #3319030: Drupal Core Roadmap for Package Manager and Update Manager 👍

Exciting that this is happening!!!!!!!! 🤩

1. Most failures look like this:
   

   Failed asserting that file "/var/www/html/core/modules/auto_updates/tests/src/Functional/../../../package_manager/tests/fixtures/release-history/drupal.9.8.2.xml" exists.
   

   That path is wrong. Because it assumes that the package_manager module still lives inside of the auto_updates module, which it does not.

2. 🤔 Lines are being moved from composer/Metapackage/PinnedDevDependencies/composer.json to composer/Metapackage/CoreRecommended/composer.json — that seems wrong?
3. Also spotted 2 harmless obsolete things in the Converter and one actual bug.

#3342120: Fix PHPStan violations (happened because PHPStan code checks stopped running…) is green and will soon be merged. That will allow you to remove the hack to avoid PHPStan complaints. 👍

Yesterday you had a proposal for how to deal with #8.1. I can't remember it though. Can you write it here?

Finally: my work on cspell in #3343430: Stop reusing core's commit-code-check.sh in favor of 4 simple commands has made it clear that we actually need two dictionary.txts, one for each module — because the core MR will be for package_maanger only.

#3342120: Fix PHPStan violations (happened because PHPStan code checks stopped running…) also landed! This now can definitely move forward.

After a failed attempt, do this in the core checkout: rm -rf core/modules/package_manager && git checkout -- core && cd core && yarn install && cd ..

Let's see if the new MR is green: #2977515-291: [ignore] Test Package manager core merge — test results at https://www.drupal.org/pift-ci-job/2608719 😬.

eagerly awaiting results of:

• https://www.drupal.org/pift-ci-job/2614581 → core + package_manager as of 1 week ago
• https://www.drupal.org/pift-ci-job/2614583 → core + package_manager as of today

👆

That takes ~1 hour to run, I’m going for a run in the mean time. Assuming that’s green, I’ll take those results and transplant them onto #3346707: Add Alpha level Experimental Package Manager module, and then we should a true green core MR! 🤓

Oops 😅 This is what I meant to do!

Awaiting resuls…

Ugh, the March 3 one passed thanks to @tedbow's fixes … but he didn't push them up to this branch! 😬 For example:

$ git show 1cb2443a09a8c6b558bdb3baeb1023b1fedef78a
commit 1cb2443a09a8c6b558bdb3baeb1023b1fedef78a
Author: Ted Bowman <ted+git@tedbow.com>
Date:   Tue Mar 7 12:32:24 2023 -0500

    Contrib: replace core_version_requirement - https://git.drupalcode.org/project/automatic_updates/-/commit/a39a545c9f9e0acf92535b7216bfd852f9f7dad1

… and on that note, it looks like he actually had already solved what I just fixed in a different way: https://git.drupalcode.org/project/drupal/-/merge_requests/3578/diffs?co... … ← also not pushed 😢

So I'm going to update the core MR at 3346707 manually, rather than spending more time on a Friday night re-creating what @tedbow already has working locally…

BTW, there are a few more things that should not be added to Drupal core, such as package_manager_update_9001(). I suggest we add the necessary

// BEGIN: DELETE FROM CORE MERGE REQUEST

and

// END: DELETE FROM CORE MERGE REQUEST

comments in this MR too, so that once this MR is merged, we're 100% satisfied with the MR it generates.

Looking great! But I don't think I agree with #18. Quoting what I wrote at #3347937-3: ComposerFixtureCreator need to move and updated for core merge request:

Are you saying it would be okay to not have the fixture creator in the MR at #3253158: Add Alpha level Experimental Update Manager module? I very much doubt that. That'd mean core requires some contrib script? 😅

IMHO this is something that is in the scope of #3341974: Finalize \Drupal\automatic_updates\Development\Converter script to update core MR to fix?

And I think this should also do #3318298: Add Composer Stager to Core vendor hardening config, since there's no point to do it as a separate issue. Or … is that because until Package Manager reaches beta-experimental, Drupal core will remove both the PM module as well as Composer Stager out of packaged builds, and that's why #3318298: Add Composer Stager to Core vendor hardening config needs to be postponed? 🤔 Unclear to me at this time.

#25: nice!

#26: once core MR is green including the fixture creator, I don't think Or could just close and re-open if/when needed is necessary — we can just commit whatever changes we need directly without the need for an issue IMHO — the commit message should just explain what we're doing. Because none of those commits would be making semantical changes to the logic of Package Manager itself, only to the script. And the script is just a tool to bring it into core, not an essential part of the module — issue queue hygiene/git archeology is pretty much non-important for it!

#27: self vs static in the context of this stand-alone script … is super unimportant IMHO 🥸

This looks great now! I trust that this results in a working core MR because adding a script cannot affect the test outcome. So … merged 🚢