Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Commit 49aa3c8

Browse files
committed
FORMS-821: Removed code analysis from workflow
1 parent f35077a commit 49aa3c8

File tree

1 file changed

+53
-53
lines changed

1 file changed

+53
-53
lines changed

.github/workflows/pr.yaml

Lines changed: 53 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -84,64 +84,64 @@ jobs:
8484
composer coding-standards-check/phpcs
8585
8686
# @todo Make this work!
87-
php-code-analysis:
88-
name: PHP code analysis
89-
runs-on: ubuntu-latest
90-
strategy:
91-
matrix:
92-
php-versions: [ '8.1' ]
93-
dependency-version: [ prefer-stable ]
94-
steps:
95-
- uses: actions/checkout@master
96-
- name: Setup PHP, with composer and extensions
97-
uses: shivammathur/setup-php@v2
98-
with:
99-
php-version: ${{ matrix.php-versions }}
100-
extensions: json, gd
101-
coverage: none
102-
tools: composer:v2
103-
# https://github.com/shivammathur/setup-php#cache-composer-dependencies
104-
- name: Get composer cache directory
105-
id: composer-cache
106-
run: echo "::set-output name=dir::$(composer config cache-files-dir)"
107-
- name: Cache dependencies
108-
uses: actions/cache@v2
109-
with:
110-
path: ${{ steps.composer-cache.outputs.dir }}
111-
key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
112-
restore-keys: ${{ runner.os }}-composer-
113-
- name: drupal-check
114-
run: |
115-
# We need a Drupal project to run drupal-check (cf. https://github.com/mglaman/drupal-check#usage)
116-
# Install Drupal
117-
composer --no-interaction create-project drupal/recommended-project:^9 drupal
118-
# Copy our module source code into the Drupal module folder.
119-
mkdir -p drupal/web/modules/contrib/os2forms_digital_post
120-
rsync --archive --compress . --exclude drupal --exclude .git drupal/web/modules/contrib/os2forms_digital_post
87+
# php-code-analysis:
88+
# name: PHP code analysis
89+
# runs-on: ubuntu-latest
90+
# strategy:
91+
# matrix:
92+
# php-versions: [ '8.1' ]
93+
# dependency-version: [ prefer-stable ]
94+
# steps:
95+
# - uses: actions/checkout@master
96+
# - name: Setup PHP, with composer and extensions
97+
# uses: shivammathur/setup-php@v2
98+
# with:
99+
# php-version: ${{ matrix.php-versions }}
100+
# extensions: json, gd
101+
# coverage: none
102+
# tools: composer:v2
103+
# # https://github.com/shivammathur/setup-php#cache-composer-dependencies
104+
# - name: Get composer cache directory
105+
# id: composer-cache
106+
# run: echo "::set-output name=dir::$(composer config cache-files-dir)"
107+
# - name: Cache dependencies
108+
# uses: actions/cache@v2
109+
# with:
110+
# path: ${{ steps.composer-cache.outputs.dir }}
111+
# key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
112+
# restore-keys: ${{ runner.os }}-composer-
113+
# - name: drupal-check
114+
# run: |
115+
# # We need a Drupal project to run drupal-check (cf. https://github.com/mglaman/drupal-check#usage)
116+
# # Install Drupal
117+
# composer --no-interaction create-project drupal/recommended-project:^9 drupal
118+
# # Copy our module source code into the Drupal module folder.
119+
# mkdir -p drupal/web/modules/contrib/os2forms_digital_post
120+
# rsync --archive --compress . --exclude drupal --exclude .git drupal/web/modules/contrib/os2forms_digital_post
121121

122-
composer --working-dir=drupal --no-interaction config minimum-stability dev
122+
# composer --working-dir=drupal --no-interaction config minimum-stability dev
123123

124-
# Allow plugins
125-
composer --working-dir=drupal config --no-plugins allow-plugins.vaimo/composer-patches true
126-
composer --working-dir=drupal config --no-plugins allow-plugins.cweagans/composer-patches true
124+
# # Allow plugins
125+
# composer --working-dir=drupal config --no-plugins allow-plugins.vaimo/composer-patches true
126+
# composer --working-dir=drupal config --no-plugins allow-plugins.cweagans/composer-patches true
127127

128-
# Add our module as a composer repository.
129-
composer --working-dir=drupal --no-interaction config --append repositories.itk-dev/os2forms_digital_post path web/modules/contrib/os2forms_digital_post
130-
# Restore Drupal composer repository.
131-
# composer --working-dir=drupal --no-interaction config --append repositories.drupal composer https://packages.drupal.org/8
128+
# # Add our module as a composer repository.
129+
# composer --working-dir=drupal --no-interaction config --append repositories.itk-dev/os2forms_digital_post path web/modules/contrib/os2forms_digital_post
130+
# # Restore Drupal composer repository.
131+
# # composer --working-dir=drupal --no-interaction config --append repositories.drupal composer https://packages.drupal.org/8
132132

133-
# Require our module.
134-
composer --working-dir=drupal --no-interaction require 'itk-dev/os2forms_digital_post:*'
133+
# # Require our module.
134+
# composer --working-dir=drupal --no-interaction require 'itk-dev/os2forms_digital_post:*'
135135

136-
# Check code
137-
# composer --working-dir=drupal --no-interaction require --dev drupal/core-dev
138-
cd drupal/web/modules/contrib/os2forms_digital_post
139-
# Remove our non-dev dependencies to prevent duplicated Drupal installation
140-
# PHP Fatal error: Cannot redeclare drupal_get_filename() (previously declared in /home/runner/work/os2forms_digital_post/os2forms_digital_post/drupal/web/modules/contrib/os2forms_digital_post/vendor/drupal/core/includes/bootstrap.inc:190) in /home/runner/work/os2forms_digital_post/os2forms_digital_post/drupal/web/core/includes/bootstrap.inc on line 190
141-
# Use sed to remove the "require" property in composer.json
142-
sed -i '/^\s*"require":/,/^\s*}/d' composer.json
143-
composer --no-interaction install
144-
composer code-analysis
136+
# # Check code
137+
# # composer --working-dir=drupal --no-interaction require --dev drupal/core-dev
138+
# cd drupal/web/modules/contrib/os2forms_digital_post
139+
# # Remove our non-dev dependencies to prevent duplicated Drupal installation
140+
# # PHP Fatal error: Cannot redeclare drupal_get_filename() (previously declared in /home/runner/work/os2forms_digital_post/os2forms_digital_post/drupal/web/modules/contrib/os2forms_digital_post/vendor/drupal/core/includes/bootstrap.inc:190) in /home/runner/work/os2forms_digital_post/os2forms_digital_post/drupal/web/core/includes/bootstrap.inc on line 190
141+
# # Use sed to remove the "require" property in composer.json
142+
# sed -i '/^\s*"require":/,/^\s*}/d' composer.json
143+
# composer --no-interaction install
144+
# composer code-analysis
145145

146146
markdown-coding-standards:
147147
runs-on: ubuntu-20.04

0 commit comments

Comments
 (0)