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

Generating dockerhub org name according to github repo owner #888

Conversation

pacostas
Copy link
Contributor

Summary

This PR adds functionality for generating Dockerhub org variable based on the repository owner name. E.x. instead of having DOCKERHUB_ORG env variable pinned to value paketobuildpacks we fetch the repository owner paketo-buildpacks or paketo-community and we remove the - to match the dockerhub org name. This change is helpful as repositories under the paketo-community organization can now sync the push-image.yml file

Use Cases

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Comment on lines 43 to 44
# Strip off the Github org prefix from repo name
# paketo-buildpacks/builder-with-some-name --> builder-with-some-name
Copy link
Member

Choose a reason for hiding this comment

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

nitpick: can we keep the comments for the registry_repo line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeap, I reverted the comment back :)

@sophiewigmore sophiewigmore enabled auto-merge (squash) February 22, 2024 21:06
@sophiewigmore sophiewigmore merged commit 6804a64 into paketo-buildpacks:main Feb 22, 2024
4 checks passed
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.

2 participants