-
Notifications
You must be signed in to change notification settings - Fork 0
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
Upgrading to CircleCI 2.0 #395
Conversation
.circleci/config.yml
Outdated
- attach_workspace: | ||
at: '~/pomegranate' | ||
- run: sudo apt-get update | ||
- run: sudo apt-get install libpq-dev libopenjp2-7-dev libmagickwand-dev imagemagick pkg-config |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any of these being installed in the v1 config, and they aren't mentioned in the README. Do we want to document additional dependencies that surfaced here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These were installed based upon my findings within other projects. I had falsely assumed them to be necessary.
config/database.yml
Outdated
database: <%= ENV["POMEGRANATE_DB"] %> | ||
host: <%= ENV["POMEGRANATE_DB_HOST"] %> | ||
username: <%= ENV["POMEGRANATE_DB_USERNAME"] %> | ||
password: <%= ENV["POMEGRANATE_DB_PASSWORD"] %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there any changes needed to run the tests locally? Could we fallback on the old defaults?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My apologies, there were errors in this configuration; I have corrected these, and can now run the tests locally without error.
0b241bc
to
f4db337
Compare
Resolves #385