Skip to content

Commit 9942d10

Browse files
authoredJul 5, 2023
Merge pull request #267 from NoelDeMartin/fix-apps-ci
Use proper npm version to run app tests
2 parents f1068ad + 3420635 commit 9942d10

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed
 

‎tests/app-setup.sh

+6-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,12 @@ then
1515

1616
if [[ $RUNTIME = ionic5 ]];
1717
then
18-
docker run --volume $basedir/app:/app --workdir /app node:14 bash -c "npm install npm@7 -g && npm ci"
18+
if [[ ! -f $basedir/app/.npmrc || -z "$(cat $basedir/app/.npmrc | grep unsafe-perm)" ]];
19+
then
20+
echo -e "\nunsafe-perm=true" >> $basedir/app/.npmrc
21+
fi
22+
23+
docker run --volume $basedir/app:/app --workdir /app node:14 bash -c "npm ci"
1924
else
2025
docker run --volume $basedir/app:/app --workdir /app node:11 npm run setup
2126
docker run --volume $basedir/app:/app --workdir /app node:11 npm ci

0 commit comments

Comments
 (0)
Please sign in to comment.