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 step to push to github package registry #332

Merged
merged 3 commits into from
Feb 14, 2020

Conversation

timfallmk
Copy link
Collaborator

Just because why not. We may use it eventually.

Copy link
Collaborator

@grampelberg grampelberg left a comment

Choose a reason for hiding this comment

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

I'm kinda meh about doing this until GPR allows for anonymous. It feels like it'll just be confusing until then. The UI isn't setup to handle PR pushes either. WDYT about limiting to release tags?

@timfallmk
Copy link
Collaborator Author

That sounds fine to me. I'm pretty sure this should only run on tags ATM. It would be nice to have some artifacts there once/if they ever "fix" that missing feature.

@grampelberg
Copy link
Collaborator

Oh, we're not actually tagging images correctly yet are we? It'd be nice to have the image tags match the release/repo tags. Maybe we should do that first?

@timfallmk
Copy link
Collaborator Author

Yeah IIRC we haven't been doing that, just the git sha to match with. Easy to update for that.

@timfallmk timfallmk merged commit 979ca44 into master Feb 14, 2020
@timfallmk timfallmk deleted the github-package-registry branch February 14, 2020 22:01
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.

None yet

2 participants