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

add build docker image #29

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

Conversation

mrchypark
Copy link

add docker image build github action.

you need to change trigger setting and image name.

I prefer triggering tags startwith v and simver like v0.0.5 and now action set 0.0.5 image tag.

And you need to set docker hub secret name DOCKERHUB_USERNAME and DOCKERHUB_TOKEN.

@salesforce-cla
Copy link

salesforce-cla bot commented May 1, 2021

Thanks for the contribution! Before we can merge this, we need @mrchypark to sign the Salesforce.com Contributor License Agreement.

@mrchypark
Copy link
Author

I signed license

@mrchypark
Copy link
Author

#10

@bman46
Copy link
Contributor

bman46 commented May 15, 2023

Any updates on this? It would be nice to have the image on dockerhub.

id: docker_meta
uses: crazy-max/[email protected]
with:
images: mrchypark/generic-sidecar-injector
Copy link
Contributor

Choose a reason for hiding this comment

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

I dont think that the main repo should have mrchypark here. Maybe use a variable/secret instead?

Copy link
Author

Choose a reason for hiding this comment

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

I would appreciate it if you could kindly provide me with the necessary repository address. I do not have authorization for the Salesforce organization. The changes I made were done using my personal account based on my needs, so please make the appropriate modifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants