-
Notifications
You must be signed in to change notification settings - Fork 15
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
[APPINT-1339] Improving Connector Framework #183
[APPINT-1339] Improving Connector Framework #183
Conversation
hhemanth
commented
Nov 19, 2018
- Made changed to Gemfile add new gems and remove some gems from production env
- Add puma configs
- Added default README
- Added circleci 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.
Looks ok so far, see minor comments.
The bin/setup
might have to be updated to work out of the box (copy the application.yml
and the database.yml
from the sample files) and to include SKIP_CONFIGURATION=true
See https://github.com/maestrano/connector-talech/commit/0a2cdb13dccc7ce1cae9ac6cc914533172ac247e
See this section of the onboarding for the commit message format
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.
LGTM apart from the question on the ruby version
363f01d
to
9e3f854
Compare
@hhemanth LGTM. could you squash those commits before I merge? Thanks. |
…ices - Made changed to Gemfile add new gems and remove some gems from production env - Add puma configs - Added default README - Added circleci config - Added setup file, which copies, database-sample.yml and application-sample.yml - concurrent-ruby 1.1.3 gem is causing issues (Nameerror), so locked it to 1.0.5 - Fixing typos - Fixing gem versions - Handling formatting of comments in entity.rb
9e3f854
to
bf7f3e1
Compare