Skip to content
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.

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

Switch to yarn #678

Merged
merged 9 commits into from
Jun 11, 2019
Merged

Switch to yarn #678

merged 9 commits into from
Jun 11, 2019

Conversation

kaustubh-nair
Copy link
Member

Fixes #351

@codecov
Copy link

codecov bot commented Jun 10, 2019

Codecov Report

Merging #678 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development     #678   +/-   ##
============================================
  Coverage        29.04%   29.04%           
============================================
  Files               34       34           
  Lines             1508     1508           
============================================
  Hits               438      438           
  Misses            1070     1070

@kaustubh-nair
Copy link
Member Author

@publiclab/mapknitter-reviewers Please review this. Thanks!

Copy link
Member

@jywarren jywarren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good! Have you tried it out on the unstable server? Once this is merged, we can turn on Dependabot for JavaScript dependencies too!

https://app.dependabot.com/accounts/publiclab/projects/23423

start.sh Outdated
@@ -3,7 +3,7 @@
pidfile=/app/tmp/pids/server.pid

bundle check || bundle install
bower install --allow-root
yarn install --allow-root
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there such flag on yarn??

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No there isn't. Removed it. Thanks 👍

@kaustubh-nair kaustubh-nair merged commit 5c0c7c7 into development Jun 11, 2019
@kaustubh-nair
Copy link
Member Author

Hi I've merged this to #669 and also published development to unstable

@alaxalves alaxalves deleted the yarn-bower branch June 11, 2019 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants