Skip to content
This repository was archived by the owner on Nov 27, 2024. It is now read-only.

ci image push#74

Merged
sadlerap merged 17 commits intokonflux-workspaces:mainfrom
filariow:ci-image-push
Apr 15, 2024
Merged

ci image push#74
sadlerap merged 17 commits intokonflux-workspaces:mainfrom
filariow:ci-image-push

Conversation

@filariow
Copy link
Member

  • update toolchain_build_push.sh script
  • update GitHub Action

Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
@filariow filariow marked this pull request as draft April 15, 2024 16:37
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Copy link
Member

@sadlerap sadlerap left a comment

Choose a reason for hiding this comment

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

A few small things, otherwise lgtm

Signed-off-by: Francesco Ilario <filario@redhat.com>
@filariow filariow marked this pull request as ready for review April 15, 2024 20:38
Signed-off-by: Francesco Ilario <filario@redhat.com>
Copy link
Member

@sadlerap sadlerap left a comment

Choose a reason for hiding this comment

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

lgtm, let me know when you think this is okay to merge

@sadlerap sadlerap merged commit f97a5b6 into konflux-workspaces:main Apr 15, 2024
@filariow filariow deleted the ci-image-push branch April 15, 2024 20:50
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.

2 participants