fix(install): Use npm-is to require Yarn before install. Fixes PWA-50…

…5. (#2384)

* fix(install): Use npm-is to require Yarn before install. Fixes PWA-505.

* Handle CI build failure

* fix: do not check in CI

* Revert docker commit and update typo

* Handle docker uid/gid switch issue

* set CI env variable in dockerfile

Co-authored-by: Devagouda <40405790+dpatil-magento@users.noreply.github.com>
Co-authored-by: Dev Patil <depatil@adobe.com>

dev.dockerfile

@@ -13,6 +13,9 @@ RUN apk --no-cache --virtual add \
     g++ \
     yarn
 
+# set env variable for CI
+ENV CI=true
+
 # copy just the dependency files and configs needed for install
 COPY packages/create-pwa/package.json ./packages/create-pwa/package.json
 COPY packages/babel-preset-peregrine/package.json ./packages/babel-preset-peregrine/package.json

package.json

@@ -30,6 +30,7 @@
     "danger": "danger-ci",
     "lint": "eslint '@(packages|scripts)/**/{*.js,package.json}' --ignore-pattern node_modules --ignore-pattern storybook-dist",
     "postbuild": "rimraf \"./packages/*/dist/{,**/}__*__\"",
+    "preinstall": "node -e 'process.env.CI||process.exit(1)' || npx npm-is yarn",
     "prepare": "node scripts/monorepo-introduction.js",
     "prettier": "prettier --loglevel warn --write '@(packages|scripts)/**/*.@(css|graphql|js)' '*.js'",
     "prettier:check": "prettier --list-different '@(packages|scripts)/**/*.@(css|graphql|js)' '*.js'",

prod.dockerfile

@@ -9,6 +9,9 @@ RUN apk --no-cache --virtual add \
     g++ \
     yarn
 
+# set env variable for CI
+ENV CI=true
+
 # copy just the dependency files and configs needed for install
 COPY packages/create-pwa/package.json ./packages/create-pwa/package.json
 COPY packages/babel-preset-peregrine/package.json ./packages/babel-preset-peregrine/package.json