smiley 1 год назад
Родитель
Сommit
b57d1703f6
2 измененных файлов с 23 добавлено и 12 удалено
  1. 5 3
      .github/PULL_REQUEST_TEMPLATE.md
  2. 18 9
      .github/workflows/ci.yml

+ 5 - 3
.github/PULL_REQUEST_TEMPLATE.md

@@ -1,11 +1,11 @@
-<!-- You can erase any parts of this template not applicable to your Pull Request. -->
-
 ## Proposed changes
 ## Proposed changes
 
 
 <!-- Describe your changes here to communicate to the maintainers why you'd like to include this pull request.
 <!-- Describe your changes here to communicate to the maintainers why you'd like to include this pull request.
 If it fixes a bug or resolves a feature request, be sure to link to that issue. -->
 If it fixes a bug or resolves a feature request, be sure to link to that issue. -->
 
 
 
 
+<!-- You can erase any of the parts below that are not applicable to your Pull Request. -->
+
 ## Types of changes
 ## Types of changes
 
 
 <!-- Put an `x` in the boxes that apply -->
 <!-- Put an `x` in the boxes that apply -->
@@ -15,6 +15,8 @@ What types of changes does your code introduce?
 - [ ] Bugfix (non-breaking change which fixes an issue)
 - [ ] Bugfix (non-breaking change which fixes an issue)
 - [ ] New feature (non-breaking change which adds functionality)
 - [ ] New feature (non-breaking change which adds functionality)
 - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
 - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
+- [ ] Documentation fix or enhancement (no code was touched)
+- [ ] Other (CI, dependencies, etc., please describe)
 
 
 
 
 ## Checklist:
 ## Checklist:
@@ -23,7 +25,7 @@ What types of changes does your code introduce?
 - [ ] I have added tests that prove my fix is effective or that my feature works
 - [ ] I have added tests that prove my fix is effective or that my feature works
 - [ ] I have added necessary documentation (if appropriate)
 - [ ] I have added necessary documentation (if appropriate)
 - [ ] Any dependent changes have been merged and published in downstream modules
 - [ ] Any dependent changes have been merged and published in downstream modules
-- [ ] Lint and unit tests pass locally with my changes
+- [ ] Static analysis and unit tests pass locally with my changes
 
 
 
 
 ## Further comments
 ## Further comments

+ 18 - 9
.github/workflows/ci.yml

@@ -1,5 +1,5 @@
-# https://help.github.com/en/categories/automating-your-workflow-with-github-actions
-# https://github.com/sebastianbergmann/phpunit/blob/master/.github/workflows/ci.yml
+# https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions
+# https://github.com/sebastianbergmann/phpunit/blob/main/.github/workflows/ci.yaml
 
 
 on:
 on:
   push:
   push:
@@ -11,6 +11,10 @@ on:
 
 
 name: "Continuous Integration"
 name: "Continuous Integration"
 
 
+env:
+  PHP_EXTENSIONS: ast, fileinfo, gd, imagick, json, mbstring
+  PHP_INI_VALUES: memory_limit=-1, error_reporting=-1, display_errors=On
+
 jobs:
 jobs:
 
 
   static-code-analysis:
   static-code-analysis:
@@ -36,11 +40,12 @@ jobs:
         uses: shivammathur/setup-php@v2
         uses: shivammathur/setup-php@v2
         with:
         with:
           php-version: ${{ matrix.php-version }}
           php-version: ${{ matrix.php-version }}
+          extensions: ${{ env.PHP_EXTENSIONS }}
+          ini-values: ${{ env.PHP_INI_VALUES }}
           coverage: none
           coverage: none
           tools: pecl
           tools: pecl
-          extensions: ast, fileinfo, gd, imagick, json, mbstring
 
 
-      - name: "Update dependencies with composer"
+      - name: "Install dependencies with composer"
         uses: ramsey/composer-install@v3
         uses: ramsey/composer-install@v3
 
 
       - name: "Run phan"
       - name: "Run phan"
@@ -70,14 +75,15 @@ jobs:
         uses: shivammathur/setup-php@v2
         uses: shivammathur/setup-php@v2
         with:
         with:
           php-version: ${{ matrix.php-version }}
           php-version: ${{ matrix.php-version }}
+          extensions: ${{ env.PHP_EXTENSIONS }}
+          ini-values: ${{ env.PHP_INI_VALUES }}
           coverage: pcov
           coverage: pcov
           tools: pecl
           tools: pecl
-          extensions: fileinfo, gd, imagick, json, mbstring
 
 
       - name: "Install dependencies with composer"
       - name: "Install dependencies with composer"
         uses: ramsey/composer-install@v3
         uses: ramsey/composer-install@v3
 
 
-      - name: "Run tests with phpunit"
+      - name: "Run tests with PHPUnit"
         run: php vendor/bin/phpunit --colors=always --configuration=phpunit.xml.dist
         run: php vendor/bin/phpunit --colors=always --configuration=phpunit.xml.dist
 
 
       - name: "Send code coverage report to Codecov.io"
       - name: "Send code coverage report to Codecov.io"
@@ -105,10 +111,11 @@ jobs:
       - name: "Install PHP"
       - name: "Install PHP"
         uses: shivammathur/setup-php@v2
         uses: shivammathur/setup-php@v2
         with:
         with:
-          php-version: "8.2"
+          php-version: "8.3"
+          extensions: ${{ env.PHP_EXTENSIONS }}
+          ini-values: ${{ env.PHP_INI_VALUES }}
           coverage: none
           coverage: none
           tools: phpDocumentor
           tools: phpDocumentor
-          extensions: fileinfo, gd, imagick, json, mbstring
 
 
       - name: "Build API docs"
       - name: "Build API docs"
         run: phpdoc --config=phpdoc.xml.dist
         run: phpdoc --config=phpdoc.xml.dist
@@ -133,7 +140,9 @@ jobs:
       - name: "Install PHP"
       - name: "Install PHP"
         uses: shivammathur/setup-php@v2
         uses: shivammathur/setup-php@v2
         with:
         with:
-          php-version: "8.2"
+          php-version: "8.3"
+          extensions: ${{ env.PHP_EXTENSIONS }}
+          ini-values: ${{ env.PHP_INI_VALUES }}
           coverage: none
           coverage: none
 
 
       - name: "Install Sphinx"
       - name: "Install Sphinx"