Enhancement: Synchronize with ergebnis/php-library-template

.gitattributes

@@ -1,12 +1,14 @@
 /.dependabot/           export-ignore
 /.github/               export-ignore
 /test/                  export-ignore
+/tools/                 export-ignore
 /.editorconfig          export-ignore
 /.gitattributes         export-ignore
 /.gitignore             export-ignore
 /.php_cs                export-ignore
 /infection.json         export-ignore
 /Makefile               export-ignore
+/phive.xml              export-ignore
 /phpstan-baseline.neon  export-ignore
 /phpstan.neon           export-ignore
 /psalm-baseline.xml     export-ignore

.github/CONTRIBUTING.md

@@ -8,9 +8,12 @@ For details, take a look at the following workflow configuration files:
 - [`workflows/prune.yaml`](workflows/prune.yaml)
 - [`workflows/release.yaml`](workflows/release.yaml)
 - [`workflows/renew.yaml`](workflows/renew.yaml)
+- [`workflows/triage.yaml`](workflows/triage.yaml)
 
 ## Coding Standards
 
+We are using [`ergebnis/composer-normalize`](https://github.com/ergebnis/composer-normalize) to normalize `composer.json`.
+
 We are using [`yamllint`](https://github.com/adrienverge/yamllint) to enforce coding standards in YAML files.
 
 If you do not have `yamllint` installed yet, run

.github/settings.yml

@@ -14,23 +14,23 @@ branches:
         required_approving_review_count: 1
       required_status_checks:
         contexts:
+          - "Code Coverage (7.4, locked)"
           - "Coding Standards (7.1, locked)"
           - "Dependency Analysis (7.4, locked)"
+          - "Mutation Tests (7.4, locked)"
           - "Static Code Analysis (7.4, locked)"
-          - "Tests (7.1, lowest)"
-          - "Tests (7.1, locked)"
           - "Tests (7.1, highest)"
-          - "Tests (7.2, lowest)"
-          - "Tests (7.2, locked)"
+          - "Tests (7.1, locked)"
+          - "Tests (7.1, lowest)"
           - "Tests (7.2, highest)"
-          - "Tests (7.3, lowest)"
-          - "Tests (7.3, locked)"
+          - "Tests (7.2, locked)"
+          - "Tests (7.2, lowest)"
           - "Tests (7.3, highest)"
-          - "Tests (7.4, lowest)"
-          - "Tests (7.4, locked)"
+          - "Tests (7.3, locked)"
+          - "Tests (7.3, lowest)"
           - "Tests (7.4, highest)"
-          - "Code Coverage (7.4, locked)"
-          - "Mutation Tests (7.4, locked)"
+          - "Tests (7.4, locked)"
+          - "Tests (7.4, lowest)"
         strict: false
       restrictions:
 

.github/workflows/integrate.yaml

@@ -135,7 +135,7 @@ jobs:
         run: "composer update --no-interaction --no-progress --no-suggest"
 
       - name: "Run maglnet/composer-require-checker"
-        uses: "docker://webfactory/composer-require-checker:2.1.0"
+        run: "tools/composer-require-checker check"
 
   static-code-analysis:
     name: "Static Code Analysis"
@@ -169,8 +169,8 @@ jobs:
         uses: "actions/cache@v1"
         with:
           path: "${{ steps.determine-composer-cache-directory.outputs.directory }}"
-          key: "${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
-          restore-keys: "${{ matrix.php-version }}-composer-locked-"
+          key: "php-${{ matrix.php-version }}-composer-${{ matrix.dependencies }}-${{ hashFiles('composer.lock') }}"
+          restore-keys: "php-${{ matrix.php-version }}-composer-${{ matrix.dependencies }}-"
 
       - name: "Install lowest dependencies from composer.json"
         if: "matrix.dependencies == 'lowest'"
@@ -424,9 +424,6 @@ jobs:
               reviewers: reviewers,
             })
 
-      - name: "Wait"
-        run: "sleep 3"
-
       - name: "Assign @ergebnis-bot"
         uses: "actions/[email protected]"
         with:
@@ -446,9 +443,6 @@ jobs:
               repo: repository.repo,
             })
 
-      - name: "Wait"
-        run: "sleep 3"
-
       - name: "Approve pull request"
         uses: "actions/[email protected]"
         if: "github.actor != 'ergebnis-bot'"
@@ -465,9 +459,6 @@ jobs:
               repo: repository.repo,
             })
 
-      - name: "Wait"
-        run: "sleep 3"
-
       - name: "Merge pull request"
         uses: "actions/[email protected]"
         with:

.github/workflows/prune.yaml

@@ -14,7 +14,7 @@ jobs:
 
     steps:
       - name: "Prune issues and pull requests"
-        uses: "actions/stale@v1"
+        uses: "actions/stale@v2"
         with:
           days-before-close: 5
           days-before-stale: 60

.github/workflows/triage.yaml

@@ -0,0 +1,42 @@
+name: "Triage"
+
+on: # yamllint disable-line rule:truthy
+  pull_request: null
+
+jobs:
+  label:
+    name: "Label"
+
+    runs-on: "ubuntu-latest"
+
+    steps:
+      - name: "Add labels based on branch name"
+        if: "github.event.action == 'opened'"
+        uses: "actions/[email protected]"
+        with:
+          github-token: "${{ secrets.ERGEBNIS_BOT_TOKEN }}"
+          script: |
+            const branchPrefixLabels = {
+              feature: "enhancement",
+              fix: "bug",
+            }
+
+            const pullRequest = context.payload.pull_request
+            const repository = context.repo
+
+            const branchName = pullRequest.head.ref
+
+            const matches = branchName.match(new RegExp('^([^/]+)\/'));
+
+            if (matches instanceof Array && branchPrefixLabels.hasOwnProperty(matches[1])) {
+              const label = branchPrefixLabels[matches[1]]
+
+              github.issues.addLabels({
+                issue_number: pullRequest.number,
+                labels: [
+                  label
+                ],
+                owner: repository.owner,
+                repo: repository.repo,
+              });
+            }

Makefile

@@ -11,14 +11,16 @@ code-coverage: vendor ## Collects coverage from running unit tests with phpunit/
 	vendor/bin/phpunit --configuration=test/Integration/phpunit.xml --coverage-text --prepend=.build/phpunit/xdebug-filter.php
 
 .PHONY: coding-standards
-coding-standards: vendor ## Fixes code style issues with friendsofphp/php-cs-fixer
+coding-standards: vendor ## Normalizes composer.json with ergebnis/composer-normalize, lints YAML files with yamllint and fixes code style issues with friendsofphp/php-cs-fixer
+	composer normalize
 	yamllint -c .yamllint.yaml --strict .
 	mkdir -p .build/php-cs-fixer
 	vendor/bin/php-cs-fixer fix --config=.php_cs --diff --diff-format=udiff --verbose
+	vendor/bin/php-cs-fixer fix --config=.php_cs.fixture --diff --diff-format=udiff --verbose
 
 .PHONY: dependency-analysis
 dependency-analysis: vendor ## Runs a dependency analysis with maglnet/composer-require-checker
-	docker run --interactive --rm --tty --volume ${PWD}:/app webfactory/composer-require-checker:2.1.0
+	tools/composer-require-checker check
 
 .PHONY: help
 help: ## Displays this list of targets with descriptions
@@ -54,4 +56,3 @@ tests: vendor ## Runs auto-review, unit, and integration tests with phpunit/phpu
 vendor: composer.json composer.lock
 	composer validate --strict
 	composer install --no-interaction --no-progress --no-suggest
-	composer normalize

README.md

@@ -275,3 +275,7 @@ Please have a look at [`LICENSE.md`](LICENSE.md).
 ## Credits
 
 The method [`FinalRule::isWhitelistedClass()`](src/Classes/FinalRule.php) is inspired by the work on [`FinalClassFixer`](https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/2.15/src/Fixer/ClassNotation/FinalClassFixer.php) and [`FinalInternalClassFixer`](https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/2.15/src/Fixer/ClassNotation/FinalInternalClassFixer.php), contributed by [Dariusz Rumiński](https://github.com/keradus), [Filippo Tessarotto](https://github.com/Slamdunk), and [Spacepossum](https://github.com/SpacePossum) for [`friendsofphp/php-cs-fixer`](https://github.com/FriendsOfPHP/PHP-CS-Fixer) (originally licensed under MIT).
+
+## Curious what I am building?
+
+:mailbox_with_mail: [Subscribe to my list](https://localheinz.com/projects/), and I will occasionally send you an email to let you know what I am working on.

phive.xml

@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phive xmlns="https://phar.io/phive">
+  <phar name="composer-require-checker" version="^2.1.0" installed="2.1.0" location="./tools/composer-require-checker" copy="true"/>
+</phive>