Clean up plugin

Setono · Aug 14, 2023 · 3c5fa7d · 3c5fa7d
1 parent 77e48e9
commit 3c5fa7d
Show file tree

Hide file tree

Showing 35 changed files with 244 additions and 411 deletions.
diff --git a/.editorconfig b/.editorconfig
@@ -15,9 +15,6 @@ indent_size = 2
 [*.md]
 trim_trailing_whitespace = false
 
-[*.sh]
-indent_style = tab
-
 [*.{yaml,yml}]
 trim_trailing_whitespace = false
 

diff --git a/.gitattributes b/.gitattributes
@@ -1,6 +1,8 @@
+/.github                        export-ignore
+/etc                            export-ignore
+/docs                           export-ignore
 /.editorconfig                  export-ignore
 /.gitattributes                 export-ignore
-/.github                        export-ignore
 /.gitignore                     export-ignore
 /README.md                      export-ignore
 /behat.yml.dist                 export-ignore
@@ -12,3 +14,5 @@
 /psalm-baseline.xml             export-ignore
 /psalm.xml                      export-ignore
 /tests                          export-ignore
+/CHANGELOG.md                   export-ignore
+/UPGRADE.md                     export-ignore
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -13,14 +13,14 @@ env:
 
 jobs:
     coding-standards:
-        name: "Coding Standards"
+        name: "Coding Standards (PHP${{ matrix.php-version }} | Deps: ${{ matrix.dependencies }})"
 
         runs-on: "ubuntu-latest"
 
         strategy:
             matrix:
                 php-version:
-                    - "7.4"
+                    - "7.4" # Always use the lowest version of PHP since a higher version could create actual syntax errors in lower versions
 
                 dependencies:
                     - "highest"
@@ -50,25 +50,40 @@ jobs:
             -   name: "Check style"
                 run: "composer check-style"
 
+            -   name: "Rector"
+                run: "vendor/bin/rector process --dry-run"
+
             -   name: "Lint yaml files"
                 run: "(cd tests/Application && bin/console lint:yaml ../../src/Resources)"
 
             -   name: "Lint twig files"
                 run: "(cd tests/Application && bin/console lint:twig ../../src/Resources)"
 
     dependency-analysis:
-        name: "Dependency Analysis"
+        name: "Dependency Analysis (PHP${{ matrix.php-version }} | Deps: ${{ matrix.dependencies }} | SF${{ matrix.symfony }})"
 
         runs-on: "ubuntu-latest"
 
         strategy:
             matrix:
                 php-version:
                     - "7.4"
+                    - "8.0"
+                    - "8.1"
+                    - "8.2"
 
                 dependencies:
+                    - "lowest"
                     - "highest"
 
+                symfony:
+                    - "^5.4"
+                    - "^6.0"
+
+                exclude:
+                    -   php-version: "7.4"
+                        symfony: "^6.0"
+
         steps:
             -   name: "Checkout"
                 uses: "actions/checkout@v3"
@@ -79,86 +94,125 @@ jobs:
                     coverage: "none"
                     extensions: "${{ env.PHP_EXTENSIONS }}"
                     php-version: "${{ matrix.php-version }}"
-                    tools: "composer-require-checker, composer-unused"
+                    tools: "composer-require-checker, composer-unused, flex"
+
+            -   name: "Remove require-dev section in composer.json"
+                run: "composer config --unset require-dev"
 
             -   name: "Install composer dependencies"
                 uses: "ramsey/composer-install@v2"
+                env:
+                    SYMFONY_REQUIRE: "${{ matrix.symfony }}"
                 with:
                     dependency-versions: "${{ matrix.dependencies }}"
 
             -   name: "Run maglnet/composer-require-checker"
-                run: "composer-require-checker check --config-file=$(pwd)/composer-require-checker.json"
+                run: "composer-require-checker check"
 
             -   name: "Run composer-unused/composer-unused"
                 run: "composer-unused"
 
     static-code-analysis:
-        name: "Static Code Analysis"
+        name: "Static Code Analysis (PHP${{ matrix.php-version }} | Deps: ${{ matrix.dependencies }} | SF${{ matrix.symfony }})"
 
         runs-on: "ubuntu-latest"
 
         strategy:
             matrix:
                 php-version:
                     - "7.4"
+                    - "8.0"
+                    - "8.1"
+                    - "8.2"
 
                 dependencies:
+                    - "lowest"
                     - "highest"
 
+                symfony:
+                    - "^5.4"
+                    - "^6.0"
+
+                exclude:
+                    -   php-version: "7.4"
+                        symfony: "^6.0"
+
         steps:
             -   name: "Checkout"
                 uses: "actions/checkout@v3"
 
             -   name: "Setup PHP, with composer and extensions"
                 uses: "shivammathur/setup-php@v2"
                 with:
-                    php-version: "${{ matrix.php-version }}"
-                    extensions: "${{ env.PHP_EXTENSIONS }}"
                     coverage: "none"
+                    extensions: "${{ env.PHP_EXTENSIONS }}"
+                    php-version: "${{ matrix.php-version }}"
+                    tools: "flex"
+
+            -   name: "Remove sylius/sylius from composer.json"
+                run: "composer remove --dev sylius/sylius --no-update --no-install"
 
             -   name: "Install composer dependencies"
                 uses: "ramsey/composer-install@v2"
+                env:
+                    SYMFONY_REQUIRE: "${{ matrix.symfony }}"
                 with:
                     dependency-versions: "${{ matrix.dependencies }}"
 
             -   name: "Static analysis"
-                run: "composer analyse"
+                run: "vendor/bin/psalm --php-version=${{ matrix.php-version }}"
 
     unit-tests:
-        name: "Unit tests"
+        name: "Unit tests (PHP${{ matrix.php-version }} | Deps: ${{ matrix.dependencies }} | SF${{ matrix.symfony }})"
 
         runs-on: "ubuntu-latest"
 
         strategy:
             matrix:
                 php-version:
                     - "7.4"
+                    - "8.0"
+                    - "8.1"
+                    - "8.2"
 
                 dependencies:
-                    - "lowest"
                     - "highest"
 
+                symfony:
+                    - "^5.4"
+                    - "^6.0"
+
+                exclude:
+                    -   php-version: "7.4"
+                        symfony: "^6.0"
+
         steps:
             -   name: "Checkout"
                 uses: "actions/checkout@v3"
 
             -   name: "Setup PHP, with composer and extensions"
                 uses: "shivammathur/setup-php@v2"
                 with:
-                    php-version: "${{ matrix.php-version }}"
-                    extensions: "${{ env.PHP_EXTENSIONS }}"
                     coverage: "none"
+                    extensions: "${{ env.PHP_EXTENSIONS }}"
+                    php-version: "${{ matrix.php-version }}"
+                    tools: "flex"
+
+            -   name: "Remove sylius/sylius from composer.json"
+                run: "composer remove --dev sylius/sylius --no-update --no-install"
 
             -   name: "Install composer dependencies"
                 uses: "ramsey/composer-install@v2"
+                env:
+                    SYMFONY_REQUIRE: "${{ matrix.symfony }}"
                 with:
                     dependency-versions: "${{ matrix.dependencies }}"
 
-            -   name: "Run phpspec"
-                run: "composer phpspec"
-
             -   name: "Run phpunit"
                 run: "composer phpunit"
+
+            -   name: "Run phpspec"
+                run: "composer phpspec"
 
     integration-tests:
         name: "Integration tests"
@@ -241,14 +295,14 @@ jobs:
                     if-no-files-found: "ignore"
 
     code-coverage:
-        name: "Code Coverage"
+        name: "Code Coverage (PHP${{ matrix.php-version }} | Deps: ${{ matrix.dependencies }})"
 
         runs-on: "ubuntu-latest"
 
         strategy:
             matrix:
                 php-version:
-                    - "7.4"
+                    - "8.2"
 
                 dependencies:
                     - "highest"

diff --git a/composer-require-checker.json b/composer-require-checker.json
diff --git a/composer.json b/composer.json
@@ -16,8 +16,13 @@
         "doctrine/persistence": "^1.3 || ^2.1",
         "league/uri": "^6.0",
         "league/uri-components": "^2.3",
-        "setono/symfony-main-request-trait": "^1.0",
+        "sylius/channel": "^1.0",
+        "sylius/channel-bundle": "^1.0",
+        "sylius/core-bundle": "^1.0",
+        "sylius/product-bundle": "^1.0",
         "sylius/resource-bundle": "^1.6",
+        "sylius/taxonomy-bundle": "^1.0",
+        "sylius/ui-bundle": "^1.0",
         "symfony/config": "^5.4 || ^6.0",
         "symfony/console": "^5.4 || ^6.0",
         "symfony/dependency-injection": "^5.4 || ^6.0",
@@ -32,6 +37,7 @@
     "require-dev": {
         "api-platform/core": "^2.7",
         "lexik/jwt-authentication-bundle": "^2.16",
+        "php-http/message-factory": "^1.1",
         "phpspec/phpspec": "^7.3",
         "phpspec/prophecy-phpunit": "^2.0",
         "phpunit/phpunit": "^9.6",
@@ -44,8 +50,7 @@
         "symfony/dotenv": "^5.4 || ^6.0",
         "symfony/intl": "^5.4 || ^6.0",
         "symfony/web-profiler-bundle": "^5.4 || ^6.0",
-        "symfony/webpack-encore-bundle": "^1.16",
-        "weirdan/doctrine-psalm-plugin": "^2.8"
+        "symfony/webpack-encore-bundle": "^1.16"
     },
     "prefer-stable": true,
     "autoload": {
@@ -65,17 +70,14 @@
         "allow-plugins": {
             "dealerdirect/phpcodesniffer-composer-installer": false,
             "ergebnis/composer-normalize": true,
+            "ocramius/package-versions": true,
             "symfony/thanks": false
         },
         "sort-packages": true
     },
     "scripts": {
-        "analyse": [
-            "@ensure-test-container-exists",
-            "psalm"
-        ],
+        "analyse": "psalm",
         "check-style": "ecs check",
-        "ensure-test-container-exists": "[[ -f tests/Application/var/cache/test/ApplicationTests_Setono_SyliusRedirectPlugin_Application_KernelTestDebugContainer.xml ]] || tests/Application/bin/console cache:warmup --env=test",
         "fix-style": "ecs check --fix",
         "phpspec": "phpspec run",
         "phpunit": "phpunit",