Skip to content
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

Lay dagger CI/CD foundations #184

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

fbegyn
Copy link
Contributor

@fbegyn fbegyn commented Apr 19, 2022

Start running CI/CD through dagger.io. Add some explantion to the README
and create a docker image for CI purposes.

Signed-off-by: Francis Begyn francis@begyn.be

Start running CI/CD through dagger.io. Add some explantion to the README
and create a docker image for CI purposes.

Signed-off-by: Francis Begyn <francis@begyn.be>
Dockerfile-ci Outdated Show resolved Hide resolved
@mcbloch mcbloch self-requested a review May 13, 2022 22:38
Copy link
Member

@mcbloch mcbloch left a comment

Choose a reason for hiding this comment

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

We probably don't want to add the whole cue.mod folder.

@fbegyn
Copy link
Contributor Author

fbegyn commented Jun 17, 2022

@mcbloch I've removed the cue.mod folder from the repository and added it to the gitignore. The CI currently fails due the codebase not passing the enforced pylint score. AFAIK this should be a good baseline for CI for this repo (and I'm sure I'll change some things once I have more insight into dagger and it CUE language.

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