Skip to content

Fix the build #5

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
Merged
merged 3 commits into from
Oct 13, 2015
Merged

Fix the build #5

merged 3 commits into from
Oct 13, 2015

Conversation

wfleming
Copy link
Contributor

  • update cc-engine-go usage: Position -> LinesOnlyPosition
  • add build script, change docker image construction
  • circle: use build script, fix fork build behavior

Will Fleming added 3 commits October 13, 2015 11:19
update cc-engine-go usage: Position -> LinesOnlyPosition
6e6f41a
add build script, change docker image construction
79d4318 
* don't build binaries locally & commit them: auto-build them
  on-demand
* keep docker image a bit smaller & don't add files it doesn't need.
circle: use build script, fix fork build behavior
4430b5b
@wfleming
Copy link
Contributor Author

This incorporates work from #4 and #3.

@wfleming
Copy link
Contributor Author

cc @codeclimate-community/cc-staff

mrb added a commit that referenced this pull request Oct 13, 2015
@mrb
Merge pull request #5 from codeclimate-community/will/build-script-an…
78d1621 
…d-circle-fix

Fix the build
@mrb mrb merged commit 78d1621 into master Oct 13, 2015
@wfleming wfleming deleted the will/build-script-and-circle-fix branch October 13, 2015 18:11
@wfleming wfleming mentioned this pull request Oct 13, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wfleming @mrb