CI: add --omit=dev and --only=prod flags to npm to work around bug with

hanging `git ls-remote`
A better solution is needed. The bug only affects node v12 and v14
This commit is contained in:
webzwo0i 2022-09-24 20:01:19 +02:00
parent 111ad5c52e
commit 5940f63fff
3 changed files with 12 additions and 4 deletions

View file

@ -78,8 +78,10 @@ jobs:
name: Install Etherpad plugins name: Install Etherpad plugins
# The --legacy-peer-deps flag is required to work around a bug in npm v7: # The --legacy-peer-deps flag is required to work around a bug in npm v7:
# https://github.com/npm/cli/issues/2199 # https://github.com/npm/cli/issues/2199
# npm v8: --omit=dev
# npm v6: --only=prod # can be dropped when support for v12/v14 is no longer needed
run: > run: >
npm install --no-save --legacy-peer-deps npm install --no-save --legacy-peer-deps --only=prod --omit=dev
ep_align ep_align
ep_author_hover ep_author_hover
ep_cursortrace ep_cursortrace
@ -166,8 +168,10 @@ jobs:
name: Install Etherpad plugins name: Install Etherpad plugins
# The --legacy-peer-deps flag is required to work around a bug in npm # The --legacy-peer-deps flag is required to work around a bug in npm
# v7: https://github.com/npm/cli/issues/2199 # v7: https://github.com/npm/cli/issues/2199
# npm v8: --omit=dev
# npm v6: --only=prod # can be dropped when support for v12/v14 is no longer needed
run: > run: >
npm install --no-save --legacy-peer-deps npm install --no-save --legacy-peer-deps --only=prod --omit=dev
ep_align ep_align
ep_author_hover ep_author_hover
ep_cursortrace ep_cursortrace

View file

@ -47,7 +47,9 @@ jobs:
# We intentionally install an old ep_align version to test upgrades to # We intentionally install an old ep_align version to test upgrades to
# the minor version number. The --legacy-peer-deps flag is required to # the minor version number. The --legacy-peer-deps flag is required to
# work around a bug in npm v7: https://github.com/npm/cli/issues/2199 # work around a bug in npm v7: https://github.com/npm/cli/issues/2199
run: npm install --no-save --legacy-peer-deps ep_align@0.2.27 # npm v8: --omit=dev
# npm v6: --only=prod # can be dropped when support for v12/v14 is no longer needed
run: npm install --no-save --legacy-peer-deps ep_align@0.2.27 --omit=dev --only=prod
# Etherpad core dependencies must be installed after installing the # Etherpad core dependencies must be installed after installing the
# plugin's dependencies, otherwise npm will try to hoist common # plugin's dependencies, otherwise npm will try to hoist common
# dependencies by removing them from src/node_modules and installing them # dependencies by removing them from src/node_modules and installing them

View file

@ -37,8 +37,10 @@ jobs:
name: Install Etherpad plugins name: Install Etherpad plugins
# The --legacy-peer-deps flag is required to work around a bug in npm # The --legacy-peer-deps flag is required to work around a bug in npm
# v7: https://github.com/npm/cli/issues/2199 # v7: https://github.com/npm/cli/issues/2199
# npm v8: --omit=dev
# npm v6: --only=prod # can be dropped when support for v12/v14 is no longer needed
run: > run: >
npm install --no-save --legacy-peer-deps npm install --no-save --legacy-peer-deps --omit=dev --only=prod
ep_align ep_align
ep_author_hover ep_author_hover
ep_cursortrace ep_cursortrace