Skip to content

Conversation

@aasorokiin
Copy link
Contributor

Update CI docker

@aasorokiin
Copy link
Contributor Author

This request has the same issue. @vegaluisjose

Copy link
Member

@vegaluisjose vegaluisjose left a comment

Choose a reason for hiding this comment

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

LGTM. After merging this one, it shouldn't fail anymore because the docker image is a newer one.

Copy link
Contributor

@tmoreau89 tmoreau89 left a comment

Choose a reason for hiding this comment

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

LGTM

@aasorokiin
Copy link
Contributor Author

Is there anything I could do to to complete this PR? @vegaluisjose

@vegaluisjose
Copy link
Member

Hey @aasorokiin ,

It looks like the CI issue is more involved than I thought and requires some work that has to be done by someone with committer privileges. Once I have the solution, I will let you know.

@vegaluisjose
Copy link
Member

Hey @aasorokiin ,

This was just addressed in #36 , so I am closing this one now.

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