Skip to content

Conversation

@leplatrem
Copy link
Contributor

Since #88 we don't build the container images anymore.

The problem with this, is that we discover container issues only when we push tags.

This pull-request is an attempt to improve that, by building the container on each pull-request.
I tried to reuse the existing Github Action, and executing the publish step only on main branch and tags. I'm not sure how to test that though.

@leplatrem leplatrem requested a review from a team as a code owner July 13, 2022 13:15
@leplatrem leplatrem temporarily deployed to cloudops July 13, 2022 13:18 Inactive
@leplatrem leplatrem temporarily deployed to cloudops July 13, 2022 14:58 Inactive
Copy link
Contributor

@bsieber-mozilla bsieber-mozilla left a comment

Choose a reason for hiding this comment

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

LGTM

@leplatrem leplatrem merged commit 795103c into main Jul 14, 2022
@leplatrem leplatrem deleted the test-build-on-pull-requests branch July 14, 2022 07:32
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