Merge pull request #35 from ensi-platform/ensitech-74

ENSITECH-74

.git_hooks/post-checkout/01-install-dependencies.sh

@@ -0,0 +1 @@
+../scripts/install-dependencies.sh

.git_hooks/post-merge/01-install-dependencies.sh

@@ -0,0 +1 @@
+../scripts/install-dependencies.sh

.git_hooks/pre-commit/01-lint-php.sh

@@ -0,0 +1 @@
+../scripts/lint-php.sh

.git_hooks/pre-commit/02-php-cs-fixer.sh

@@ -0,0 +1 @@
+../scripts/php-cs-fixer.sh

.git_hooks/pre-push/01-composer-validate.sh

@@ -0,0 +1 @@
+../scripts/composer-validate.sh

.git_hooks/pre-push/02-phpstan.sh

@@ -0,0 +1 @@
+../scripts/phpstan.sh

.git_hooks/pre-push/03-test-code.sh

@@ -0,0 +1 @@
+../scripts/test-code.sh

.git_hooks/pre-push/composer-validate.sh → .git_hooks/scripts/composer-validate.sh

@@ -1,16 +1,14 @@
 #!/bin/bash
 
-source $(pwd)/.git_hooks/external_runtime
-
-# validate composer 
+# Validate composer.json before commit
 
 ESC_SEQ="\x1b["
 COL_RESET=$ESC_SEQ"39;49;00m"
 COL_RED=$ESC_SEQ"0;31m"
 COL_GREEN=$ESC_SEQ"0;32m"
 COL_YELLOW=$ESC_SEQ"0;33m"
 
-echo 
+echo
 printf "$COL_YELLOW%s$COL_RESET\n" "Running pre-push hook: \"composer-validate\""
 
 VALID=$(composer validate --strict --no-check-publish --no-check-all | grep "is valid")

.git_hooks/scripts/install-dependencies.sh

@@ -0,0 +1,18 @@
+#!/bin/bash
+
+# - 'composer update' if changed composer.json
+
+ESC_SEQ="\x1b["
+COL_RESET=$ESC_SEQ"39;49;00m"
+COL_RED=$ESC_SEQ"0;31m"
+COL_GREEN=$ESC_SEQ"0;32m"
+COL_YELLOW=$ESC_SEQ"0;33m"
+
+changed_files="$(git diff-tree -r --name-only --no-commit-id HEAD@{1} HEAD)"
+
+check_run() {
+    echo "$changed_files" | grep -q "$1" && echo " * changes detected in $1" && echo " * running $2" && eval "$2"
+}
+
+check_run composer.json "composer update"
+exit 0

.git_hooks/pre-commit/lint-php.sh → .git_hooks/scripts/lint-php.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
 
-source $(pwd)/.git_hooks/external_runtime
+# Lint all added php-files via 'php -l'
 
 ROOT_DIR="$(pwd)/"
 LIST=$(git diff-index --cached --name-only --diff-filter=ACMR HEAD)
@@ -33,13 +33,13 @@ do
     fi
 done
 if [ "$ERRORS_BUFFER" != "" ]; then
-    echo 
+    echo
     echo "These errors were found in try-to-commit files: "
     echo -e $ERRORS_BUFFER
-    echo 
+    echo
     printf "$COL_RED%s$COL_RESET\r\n\r\n" "Can't commit, fix errors first."
     exit 1
 else
     echo "Okay"
     exit 0
-fi
+fi

.git_hooks/pre-commit/php-cs-fixer.sh → .git_hooks/scripts/php-cs-fixer.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
 
-source $(pwd)/.git_hooks/external_runtime
+# Check code style via '.php-cs-fixer.php'
 
 EXECUTABLE_NAME=php-cs-fixer
 EXECUTABLE_COMMAND=fix

.git_hooks/scripts/phpstan.sh

@@ -0,0 +1,20 @@
+#!/bin/bash
+
+# Run composer phpstan
+
+ESC_SEQ="\x1b["
+COL_RESET=$ESC_SEQ"39;49;00m"
+COL_RED=$ESC_SEQ"0;31m"
+COL_GREEN=$ESC_SEQ"0;32m"
+COL_YELLOW=$ESC_SEQ"0;33m"
+
+echo
+printf "$COL_YELLOW%s$COL_RESET\n" "Running pre-push hook: \"phpstan\""
+
+if composer phpstan; then
+    echo "Okay"
+    exit 0
+else
+    printf "$COL_RED%s$COL_RESET\r\n" "phpstan analysis failed."
+    exit 1
+fi

.git_hooks/pre-push/test-code.sh → .git_hooks/scripts/test-code.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
 
-source $(pwd)/.git_hooks/external_runtime
+# Run composer test
 
 ESC_SEQ="\x1b["
 COL_RESET=$ESC_SEQ"39;49;00m"

.github/CONTRIBUTING.md

@@ -0,0 +1,53 @@
+# Contributing
+
+Contributions are **welcome** and will be fully **credited**.
+
+Please read and understand the contribution guide before creating an issue or pull request.
+
+## Etiquette
+
+This project is open source, and as such, the maintainers give their free time to build and maintain the source code
+held within. They make the code freely available in the hope that it will be of use to other developers. It would be
+extremely unfair for them to suffer abuse or anger for their hard work.
+
+Please be considerate towards maintainers when raising issues or presenting pull requests. Let's show the
+world that developers are civilized and selfless people.
+
+It's the duty of the maintainer to ensure that all submissions to the project are of sufficient
+quality to benefit the project. Many developers have different skillsets, strengths, and weaknesses. Respect the maintainer's decision, and do not be upset or abusive if your submission is not used.
+
+## Viability
+
+When requesting or submitting new features, first consider whether it might be useful to others. Open
+source projects are used by many developers, who may have entirely different needs to your own. Think about
+whether or not your feature is likely to be used by other users of the project.
+
+## Procedure
+
+Before filing an issue:
+
+- Attempt to replicate the problem, to ensure that it wasn't a coincidental incident.
+- Check to make sure your feature suggestion isn't already present within the project.
+- Check the pull requests tab to ensure that the bug doesn't have a fix in progress.
+- Check the pull requests tab to ensure that the feature isn't already in progress.
+
+Before submitting a pull request:
+
+- Check the codebase to ensure that your feature doesn't already exist.
+- Check the pull requests to ensure that another person hasn't already submitted the feature or fix.
+
+## Requirements
+
+If the project maintainer has any additional requirements, you will find them listed here.
+
+- **Add tests!** - Your patch won't be accepted if it doesn't have tests.
+
+- **Document any change in behaviour** - Make sure the `README.md` and any other relevant documentation are kept up-to-date.
+
+- **Consider our release cycle** - We try to follow [SemVer v2.0.0](https://semver.org/). Randomly breaking public APIs is not an option.
+
+- **One pull request per feature** - If you want to do more than one thing, send multiple pull requests.
+
+- **Send coherent history** - Make sure each individual commit in your pull request is meaningful. If you had to make multiple intermediate commits while developing, please [squash them](https://www.git-scm.com/book/en/v2/Git-Tools-Rewriting-History#Changing-Multiple-Commit-Messages) before submitting.
+
+**Happy coding**!

.github/SECURITY.md

@@ -0,0 +1,3 @@
+# Security Policy
+
+If you discover any security related issues, please email [email protected] instead of using the issue tracker.

.github/workflows/run-tests.yml

@@ -13,22 +13,8 @@ jobs:
       fail-fast: true
       matrix:
         php: [8.1, 8.2, 8.3]
-        laravel: [8.*, 9.*, 10.*, 11.*]
-        include:
-          - laravel: 8.*
-            testbench: ^6.23
-          - laravel: 9.*
-            testbench: 7.*
-          - laravel: 10.*
-            testbench: 8.*
-          - laravel: 11.*
-            testbench: 9.*
+        laravel: [9.*, 10.*, 11.*]
         exclude:
-          - laravel: 10.*
-            php: 8.0
-
-          - laravel: 11.*
-            php: 8.0
           - laravel: 11.*
             php: 8.1
 
@@ -52,7 +38,14 @@ jobs:
           echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"
       - name: Install dependencies
         run: |
-          composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --no-interaction --no-update
+          composer require "laravel/framework:${{ matrix.laravel }}" --no-interaction --no-update
           composer update --prefer-stable --prefer-dist --no-interaction
+
+      - name: Composer Validate
+        run: ./.git_hooks/scripts/composer-validate.sh
+
       - name: Execute tests
-        run: vendor/bin/pest
+        run: composer test-ci
+
+      - name: Execute phpstan
+        run: ./.git_hooks/scripts/phpstan.sh