Skip to content

Commit

Permalink
fix(php-idna and php-sdk workflows): reviewed coverage report generat…
Browse files Browse the repository at this point in the history
…ion implementation
  • Loading branch information
AsifNawaz-cnic committed Mar 22, 2024
1 parent df40c79 commit cc413c5
Show file tree
Hide file tree
Showing 4 changed files with 25 additions and 5 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/php-idna-translator-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,19 @@ jobs:
uses: shivammathur/setup-php@v2
with:
php-version: ${{ vars.RTLDEV_MW_CI_PHP_VERSION }}
coverage: xdebug
- name: Install dependencies
run: |
npm ci
echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
composer update --no-dev
- name: PHP Class Documentation
run: composer run-script docs
- name: Code Coverage Report
run: composer run-script coverage
- name: Release
env:
GITHUB_TOKEN: ${{ secrets.RTLDEV_MW_CI_TOKEN }}
TEAMS_NOTIFICATION_URI: ${{ secrets.RTLDEV_MW_CI_NOTIFICATION_URI }}
COMMIT_SHA: ${{ github.sha }}
run: npx semantic-release
run: npx semantic-release
12 changes: 9 additions & 3 deletions .github/workflows/php-idna-translator-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ jobs:
- name: Install Dependencies::PHP${{ matrix.php-version }}
run: |
echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
composer update
composer update
- name: Super Linter Code Base
uses: github/super-linter/slim@v5
env:
Expand All @@ -44,9 +45,14 @@ jobs:
with:
fetch-depth: 0
persist-credentials: false
- name: Install PHP ${{ matrix.php-version }}
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php-version }}
coverage: xdebug
- name: Install Dependencies::PHP${{ matrix.php-version }}
run: |
echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
composer update
- name: Code Validation
run: composer run-script test
- name: Code Validation & Coverage
run: composer run-script coverage
6 changes: 5 additions & 1 deletion .github/workflows/php-sdk-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,16 @@ jobs:
uses: shivammathur/setup-php@v2
with:
php-version: ${{ vars.RTLDEV_MW_CI_PHP_VERSION }}
coverage: xdebug
- name: Install dependencies
run: |
npm ci
echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
composer update --no-dev
composer run-script docs
- name: PHP Class Documentation
run: composer run-script docs
- name: Code Coverage Report
run: composer run-script test
- name: Release
env:
GITHUB_TOKEN: ${{ secrets.RTLDEV_MW_CI_TOKEN }}
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/php-sdk-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,11 @@ jobs:
with:
fetch-depth: 0
persist-credentials: false
- name: Install PHP ${{ matrix.php-version }}
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php-version }}
coverage: xdebug
- name: Install Dependencies::PHP${{ matrix.php-version }}
run: |
echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
Expand Down

0 comments on commit cc413c5

Please sign in to comment.