Merge pull request #520 from PHPCSStandards/feature/ghactions-various-tweaks

GH Actions: various tweaks

Author: jrfnl

.github/workflows/basics.yml

@@ -62,7 +62,8 @@ jobs:
 
       # Show XML violations inline in the file diff.
       # @link https://github.com/marketplace/actions/xmllint-problem-matcher
-      - uses: korelstar/xmllint-problem-matcher@v1
+      - name: Enable showing XML issues inline
+        uses: korelstar/xmllint-problem-matcher@v1
 
       # Validate the XML file.
       # @link http://xmlsoft.org/xmllint.html
@@ -99,7 +100,7 @@ jobs:
 
       # Install dependencies and handle caching in one go.
       # Dependencies need to be installed to make sure the PHPCS and PHPUnit classes are recognized.
-      # @link https://github.com/marketplace/actions/install-composer-dependencies
+      # @link https://github.com/marketplace/actions/install-php-dependencies-with-composer
       - name: Install Composer dependencies
         uses: "ramsey/composer-install@v2"
         with:

.yamllint.yml

@@ -6,6 +6,7 @@ yaml-files:
   - '*.yaml'
   - '*.yml'
   - '.yamllint'
+  - 'phpstan.neon*'
 
 # Rule documentation: https://yamllint.readthedocs.io/en/stable/rules.html
 rules:

phpstan.neon.dist

@@ -1,5 +1,5 @@
 parameters:
-    #phpVersion: 50400 # Needs to be 70100 or higher... sigh...
+    # phpVersion: 50400 # Needs to be 70100 or higher... sigh...
     level: 6
     paths:
         - phpcsutils-autoload.php
@@ -14,6 +14,8 @@ parameters:
     treatPhpDocTypesAsCertain: false
 
     ignoreErrors:
+        # yamllint disable rule:line-length
+
         # Level 0
         # Ignoring this as availability depends on which PHPUnit Polyfills version is loaded/installed. This is 100% okay.
         -
@@ -72,10 +74,10 @@ parameters:
         -
             message: '`^Parameter #[0-9]+ \$\S+ of static method PHPCSUtils\\(?!Tests)[A-Za-z]+\\[A-Za-z]+::[A-Za-z]+\(\) expects [^,]+, \S+ given\.$`'
             paths:
-              - Tests/BackCompat/BCFile/GetTokensAsStringTest.php
-              - Tests/Exceptions/TestTargetNotFound/TestTargetNotFoundTest.php
-              - Tests/Fixers/SpacesFixer/SpacesFixerExceptionsTest.php
-              - Tests/Utils/GetTokensAsString/GetTokensAsStringTest.php
+                - Tests/BackCompat/BCFile/GetTokensAsStringTest.php
+                - Tests/Exceptions/TestTargetNotFound/TestTargetNotFoundTest.php
+                - Tests/Fixers/SpacesFixer/SpacesFixerExceptionsTest.php
+                - Tests/Utils/GetTokensAsString/GetTokensAsStringTest.php
 
         # Ignoring as this is fine.
         -
@@ -93,3 +95,5 @@ parameters:
         -
             message: '`^Method PHPCSUtils\\Tests\\[^: ]+Test(Case)?::\S+\(\) has parameter \$\S* with no value type specified in iterable type array\.$`'
             path: Tests/*
+
+            # yamllint enable rule:line-length