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

CENG-260: update-cloudsmith-cli-to-include-zipapp-in-the-release #164

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

BartoszBlizniak
Copy link
Collaborator

  • Added zipapp of the cloudsmith-cli
  • Added workflow to generate the release with .pyz file

Copy link
Contributor

@paddycarey paddycarey left a comment

Choose a reason for hiding this comment

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

Looks like it does what it says on the tin! Few questions just.

@@ -0,0 +1,49 @@
name: Create Zipapp and Release
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we do this in CircleCI since that's where the rest of the CI for this repo already runs?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm not that familiar with CircleCi so decided to do it in GHA instead. I did briefly take a look at it but GHA has a nicer and easier way of implementing this as part of their built-in action and uses pre-defined credentials. If this is a biggie, I can take a look at it but if you're happy to roll with this and I can take a proper look at this when I get some more time.

@paddycarey paddycarey merged commit 428f567 into cloudsmith-io:master Jun 11, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants