Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Initial implementation of circle job. #7

Closed
wants to merge 3 commits into from
Closed

Initial implementation of circle job. #7

wants to merge 3 commits into from

Conversation

jim
Copy link

@jim jim commented Aug 3, 2020

This adds a sample CI configuration to the project.

Closes #4

@jim jim requested review from eeeady and mikena-truss August 3, 2020 17:36
@jim jim self-assigned this Aug 3, 2020
Copy link

@eeeady eeeady left a comment

Choose a reason for hiding this comment

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

I have an in-line question and... I was wondering how you consume the test results. We have some examples for how to get JUNIT reports out and into circleci.

- restore_cache:
keys:
- go-mod-sources-v1-{{ checksum "go.sum" }}
- run: wget -qO- https://repo1.maven.org/maven2/org/flywaydb/flyway-commandline/6.5.3/flyway-commandline-6.5.3-linux-x64.tar.gz | tar xvz && sudo ln -s `pwd`/flyway-6.5.3/flyway /usr/local/bin
Copy link

Choose a reason for hiding this comment

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

Is this a common utility we're using? It might be worth putting this in a docker container for usage.

Copy link
Author

Choose a reason for hiding this comment

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

There is a docker image that we can probably leverage here, although I haven't looked into how to do that in Circle yet. Do you think that is a better approach than installing it into this image?

@jim
Copy link
Author

jim commented Aug 4, 2020

@eeeady I will add the Junit test result export to this.

@jim jim closed this Sep 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup CI
2 participants