Convert profile hooks

Appears to be 1 test profile "testing_hooks" with a profile file that could probably be converted here too thoughts?

That profile is explicitly testing that procedural hooks are picked up.

In that case this looks fine.

Committed e1e072f and pushed to 11.x. Thanks!

The phpstan baseline has been manually edited - please don't do that. PHPStan should be the only thing that generates it.

diff --git a/core/.phpstan-baseline.php b/core/.phpstan-baseline.php
index 90302bc1c62..a4b7ee050fb 100644
--- a/core/.phpstan-baseline.php
+++ b/core/.phpstan-baseline.php
@@ -54604,7 +54604,6 @@
 	'path' => __DIR__ . '/profiles/minimal/tests/src/Functional/MinimalTest.php',
 ];
 $ignoreErrors[] = [
-	// identifier: missingType.return
 	'message' => '#^Method Drupal\\\\Tests\\\\standard\\\\Functional\\\\StandardTest\\:\\:assertConfigSchema\\(\\) has no return type specified\\.$#',
 	'identifier' => 'missingType.return',
 	'count' => 1,

Fixed on commit by running vendor/bin/phpstan analyse -c core/phpstan.neon.dist --generate-baseline=core/.phpstan-baseline.php

Thanks! I'll use that command, I only really used that for bulk changes so there are quite a few issues out there with manual updates, but I can update them as I come across them.

For some reason, I'm getting occasional PHP warnings after this change:

Warning: include_once(/var/www/html/drupal/core/profiles/standard/standard.profile): Failed to open stream: No such file or directory in include_once() (line 160 of /var/www/html/drupal/core/lib/Drupal/Core/Extension/Extension.php)

Can confirm I saw that this too this morning after doing drush cr. Didn't know where it came from thanks @lauriii

On new or existing sites?

For me it was an existing site locally. I pulled latest changes and ran drush cr and got the warning.

Second cr no warning

Yeah that will probably happen for each module or profile file that disappears.