Skip to content

Commit

Permalink
- remove code from docker/testing.entrypoint.sh that install npm depe…
Browse files Browse the repository at this point in the history
…ndencies. The npm install is now triggered as the post-install-cmd in composer.json
  • Loading branch information
jasonbahl committed Mar 8, 2022
1 parent bf18815 commit 8fcb6db
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 12 deletions.
2 changes: 2 additions & 0 deletions docker/app.post-setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,5 @@ if $( wp maintenance-mode is-active --allow-root ); then
echo "Deactivating maintenance mode"
wp maintenance-mode deactivate --allow-root
fi


14 changes: 2 additions & 12 deletions docker/testing.entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -83,23 +83,13 @@ if [ ! -f "$PROJECT_DIR/c3.php" ]; then
curl -L 'https://raw.github.com/Codeception/c3/2.0/c3.php' > "$PROJECT_DIR/c3.php"
fi

# Install PHP dependencies

# Install the PHP dependencies
echo "Running composer update"
COMPOSER_MEMORY_LIMIT=-1 composer update
echo "Running composer install"
COMPOSER_MEMORY_LIMIT=-1 composer install --no-interaction

## Install JavaScript Dependencies
if [ -f package.json ]; then
echo "Running npm install"
npm ci

## Build the JavaScript app
echo "Building the JavaScript app"
npm build
else
echo "Skipping npm install"
fi

# Install pcov/clobber if PHP7.1+
if version_gt $PHP_VERSION 7.0 && [[ -n "$COVERAGE" ]] && [[ -z "$USING_XDEBUG" ]]; then
Expand Down

0 comments on commit 8fcb6db

Please sign in to comment.