New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use Yarn to manage dependencies on CI #576

Merged

ashmaroli merged 1 commit into jekyll:master from ashmaroli:ci-yarn

Dec 30, 2019

Member

ashmaroli commented Dec 30, 2019

Since we already check-in yarn.lock to the repository, it would be better if CI builds used the same set.


          Use Yarn to manage dependencies

03058ad

ashmaroli marked this pull request as ready for review

December 30, 2019 07:25

ashmaroli merged commit 7d6868c into jekyll:master

ashmaroli deleted the ci-yarn branch

December 30, 2019 07:26

mertkahyaoglu reviewed

View reviewed changes

script/bootstrap

@@ @@ -4,4 +4,4 @@ set -e @@
               script/branding
               bundle install
-              npm install
+              yarn install

Member

mertkahyaoglu Dec 30, 2019

FYI: just yarn was enough 😄 It can stay like this though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment