Skip to content

Commit 5aec047

Browse files
committed
Some minor improvements for setup script
Since we use `yarn` for our container setup and try to enforce dependencies, we should also use yarn in the setup script. Signed-off-by: Sheogorath <[email protected]>
1 parent f27e11a commit 5aec047

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

bin/setup

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,12 @@ if [ -d .git ]; then
88
cd "$(git rev-parse --show-toplevel)"
99
fi
1010

11-
if ! type npm > /dev/null
11+
if ! type yarn > /dev/null
1212
then
1313
cat << EOF
14-
npm is not installed, please install Node.js and npm.
14+
yarn is not installed, please install Node.js, npm and yarn.
1515
Read more on Node.js official website: https://nodejs.org
16+
And for yarn package manager at: https://yarnpkg.com/en/
1617
Setup will not be run
1718
EOF
1819
exit 0
@@ -27,8 +28,9 @@ if [ ! -f .sequelizerc ]; then
2728
cp .sequelizerc.example .sequelizerc
2829
fi
2930

30-
echo "install npm packages"
31-
BUILD_ASSETS=false npm install
31+
echo "install packages"
32+
yarn install --pure-lockfile
33+
yarn install --production=false --pure-lockfile
3234

3335
cat << EOF
3436

0 commit comments

Comments
 (0)