Remove psr/container and add psr/event-dispatcher dependency (#167)

Author: vjik

.github/workflows/dependency.yml

@@ -0,0 +1,68 @@
+on:
+  pull_request:
+    paths-ignore:
+      - 'docs/**'
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - '.gitignore'
+      - '.gitattributes'
+      - 'infection.json.dist'
+      - 'phpunit.xml.dist'
+      - 'psalm.xml'
+
+  push:
+    paths-ignore:
+      - 'docs/**'
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - '.gitignore'
+      - '.gitattributes'
+      - 'infection.json.dist'
+      - 'phpunit.xml.dist'
+      - 'psalm.xml'
+
+name: dependency checker
+
+jobs:
+  mutation:
+    name: PHP ${{ matrix.php }}-${{ matrix.os }}
+
+    runs-on: ${{ matrix.os }}
+
+    strategy:
+      matrix:
+        os:
+          - ubuntu-latest
+
+        php:
+          - 8.1
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Install PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          tools: composer:v2, cs2pr
+          coverage: none
+
+      - name: Determine composer cache directory
+        run: echo "COMPOSER_CACHE_DIR=$(composer config cache-dir)" >> $GITHUB_ENV
+
+      - name: Cache dependencies installed with composer
+        uses: actions/cache@v3
+        with:
+          path: ${{ env.COMPOSER_CACHE_DIR }}
+          key: php${{ matrix.php }}-composer-${{ hashFiles('**/composer.json') }}
+          restore-keys: |
+            php${{ matrix.php }}-composer-
+      - name: Update composer
+        run: composer self-update
+
+      - name: Install dependencies with composer
+        run: composer update --prefer-dist --no-interaction --no-progress --optimize-autoloader --ansi
+
+      - name: Check dependency
+        run: vendor/bin/composer-require-checker

CHANGELOG.md

@@ -3,6 +3,8 @@
 ## 1.1.0 under development
 
 - Enh #163: Allow multiple separate hosts with new `Route::hosts()` method (@Gerych1984)
+- Chg #167: Move `psr/container` dependency to dev requirements (@vjik)
+- Chg #167: Add `psr/event-dispatcher` dependency (@vjik)
 - Enh #168: Allow multiple separate hosts with new `Group::hosts()` method (@rustamwin)
 
 ## 1.0.0 December 30, 2021

composer.json

@@ -19,7 +19,7 @@
     },
     "require": {
         "php": "^7.4|^8.0",
-        "psr/container": "^1.0",
+        "psr/event-dispatcher": "^1.0",
         "psr/http-factory": "^1.0",
         "psr/http-message": "^1.0",
         "psr/http-server-handler": "^1.0",
@@ -29,15 +29,16 @@
         "yiisoft/router-implementation": "1.0.0"
     },
     "require-dev": {
-        "nyholm/psr7": "^1.3",
+        "maglnet/composer-require-checker": "^3.8",
+        "nyholm/psr7": "^1.5",
         "phpunit/phpunit": "^9.5",
-        "psr/event-dispatcher": "^1.0",
-        "roave/infection-static-analysis-plugin": "^1.16",
+        "psr/container": "^1.1|^2.0",
+        "roave/infection-static-analysis-plugin": "^1.18",
         "spatie/phpunit-watcher": "^1.23",
-        "vimeo/psalm": "^4.18",
+        "vimeo/psalm": "^4.23",
         "yiisoft/di": "^1.0",
         "yiisoft/dummy-provider": "^1.0.0",
-        "yiisoft/test-support": "^1.3"
+        "yiisoft/test-support": "^1.4"
     },
     "autoload": {
         "psr-4": {

dependency-checker.json

@@ -0,0 +1,5 @@
+{
+    "scan-files": [
+        "src/*.php"
+    ]
+}

tests/Support/Container.php

@@ -24,7 +24,7 @@ public function get($id)
         throw new NotFoundException("$id was not found in container");
     }
 
-    public function has($id)
+    public function has(string $id): bool
     {
         return array_key_exists($id, $this->instances);
     }