Change ordering of tasks, add separate PHPStan jobs

grumphp.yml

@@ -1,6 +1,6 @@
 
 parameters:
-    convention.EXEC_GRUMPHP_COMMAND: 'docker-compose --env-file .env.test -f docker-compose.yml -f docker-compose.test.yml exec -T php'
+    convention.EXEC_GRUMPHP_COMMAND: 'docker run -i --rm --name my-running-script -v "$PWD":/app -w /app php:8.1-cli'
     convention.git_blacklist.keywords: [ 'die\\(', 'dd\\(', 'dump\\(', 'var_dump\\(', 'exit;' ]
     convention.git_branch_name.blacklist: [ master, main ]
     convention.gitlab_lint.api_token: 'MY TOKEN'
@@ -13,8 +13,8 @@ grumphp:
     stop_on_failure: true
 
     tasks:
-        composer: null
-        composer_normalize: null
+        composer: ~      # Runs "composer validate" to check the file when it's been changed
+        composer_normalize: ~
         composer_require_checker: ~
         #deptrac: ~
         doctrine_schema_check:
@@ -32,21 +32,24 @@ grumphp:
         git_commit_message: ~
         gitlab_lint:
             api_token: '%convention.gitlab_lint.api_token%'
-        phpstan:
-            configuration: phpstan.neon
-            use_grumphp_paths: false # Might want to remove this after testing is finished, to speed up processing
-        phpunitbridge:
-            testsuite: Unit
-        phpunitbridge_all:
-            metadata: { task: phpunitbridge }
-        symfony_lint_twig:
+        lint_twig:
             scripts: [["-c", "./bin/console lint:twig templates"]]
             triggered_by: [ twig ]
             metadata: { task: shell }
-        symfony_lint_yaml:
+        lint_yaml:
             scripts: [[ "-c", "./bin/console lint:yaml src config tests translations" ]]
             triggered_by: [ yaml ]
             metadata: { task: shell }
+        phpstan:
+            configuration: phpstan.neon
+        phpstan_all:
+            configuration: phpstan.neon
+            use_grumphp_paths: false
+            metadata: { task: phpstan }
+        phpunitbridge:
+            testsuite: Unit
+        phpunitbridge_all:
+            metadata: { task: phpunitbridge }
 
     extensions:
         - Headsnet\GrumPHP\GitlabLint\Loader
@@ -57,17 +60,18 @@ grumphp:
                 - git_commit_message
         git_pre_commit:
             tasks:
-                - composer
-                - composer_normalize
+                - git_branch_name # This should go first for maximum fail-fast
                 - ecs
-                - git_branch_name
-                - gitlab_lint
                 - phpstan
                 - phpunitbridge
-                - symfony_lint_twig
-                - symfony_lint_yaml
+                - composer
+                - composer_normalize
+                - lint_twig
+                - lint_yaml
         my_pre_push:
             tasks:
-                #- deptrac
+                - phpstan_all
+                - gitlab_lint
                 - doctrine_schema_check
+                #- deptrac
                 - phpunitbridge_all