Skip to content

Adds gomoderator docker to travis build#129

Merged
yaoandrew merged 5 commits intostagingfrom
vdb-1198-gomoderator-travis
Mar 27, 2020
Merged

Adds gomoderator docker to travis build#129
yaoandrew merged 5 commits intostagingfrom
vdb-1198-gomoderator-travis

Conversation

@yaoandrew
Copy link
Copy Markdown
Contributor

@yaoandrew yaoandrew commented Feb 18, 2020

This PR contains a Dockerfile that is used in the .travis.yml. It should run gomoderator across both repos. Exit codes have been updated to be more explicit in this PR:
sky-ecosystem/vulcanizedb#49

@yaoandrew yaoandrew force-pushed the vdb-1198-gomoderator-travis branch from 55ce869 to 81cef62 Compare February 19, 2020 19:45
@yaoandrew yaoandrew changed the title Adds vulcanizedb to travis build Adds gomoderator docker to travis build Feb 19, 2020
@yaoandrew yaoandrew force-pushed the vdb-1198-gomoderator-travis branch from 81cef62 to 6517e3e Compare February 19, 2020 23:31
@yaoandrew yaoandrew force-pushed the vdb-1198-gomoderator-travis branch 2 times, most recently from 87ad456 to 1ec37e9 Compare February 25, 2020 23:14
@yaoandrew yaoandrew force-pushed the vdb-1198-gomoderator-travis branch from 7b9cf68 to 1205734 Compare March 5, 2020 03:50

WORKDIR /go/src/github.com/makerdao
RUN git clone https://github.com/makerdao/vulcanizedb.git
RUN git clone https://github.com/makerdao/vdb-mcd-transformers.git
Copy link
Copy Markdown
Contributor

@rmulhol rmulhol Mar 5, 2020

Choose a reason for hiding this comment

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

I don't think we need to clone this repo (and probably don't want to so we avoid being on the wrong branch).

Instead maybe we can just COPY . . (may want to enter the appropriate workdir first, e.g. just do it after line 14)

@yaoandrew yaoandrew force-pushed the vdb-1198-gomoderator-travis branch from 6963621 to 5182f48 Compare March 26, 2020 22:53
@yaoandrew yaoandrew requested a review from rmulhol March 26, 2020 23:08
Copy link
Copy Markdown
Contributor

@rmulhol rmulhol left a comment

Choose a reason for hiding this comment

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

:shipit:

@yaoandrew yaoandrew merged commit 638dc09 into staging Mar 27, 2020
@yaoandrew yaoandrew deleted the vdb-1198-gomoderator-travis branch March 27, 2020 17:49
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