Skip to content

Override CircleCI's dependencies command#297

Merged
tysongach merged 1 commit intomasterfrom
tg-circle
Oct 27, 2017
Merged

Override CircleCI's dependencies command#297
tysongach merged 1 commit intomasterfrom
tg-circle

Conversation

@tysongach
Copy link
Contributor

CircleCI was inferring that npm install needed to be ran for project
dependencies. This isn't case, as the Node dependencies are only used
for spinning up a local development instance, not for tests.

Running npm install was causing a failing build due to some issues
in the packages: https://circleci.com/gh/thoughtbot/bitters/161

This change manually overrides CircleCI's inference, by telling it to
only install the needed Ruby dependencies for testing.

Circle docs: https://circleci.com/docs/1.0/configuration/
Failed

CircleCI was inferring that `npm install` needed to be ran for project
dependencies. This isn't case, as the Node dependencies are only used
for spinning up a local development instance, not for tests.

Running `npm install` was causing a failing build due to some issues
in the packages: https://circleci.com/gh/thoughtbot/bitters/161

This change manually overrides CircleCI's inference, by telling it to
only install the needed Ruby dependencies for testing.

Circle docs: https://circleci.com/docs/1.0/configuration/
Failed
Copy link

@wardpenney wardpenney left a comment

Choose a reason for hiding this comment

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

Nice catch 🎉

@tysongach tysongach merged commit 6a85e70 into master Oct 27, 2017
@tysongach tysongach deleted the tg-circle branch October 27, 2017 14:24
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.

2 participants