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

Publish using GitHub releases #4

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

CoffeeCoder1
Copy link
Collaborator

Use GitHub releases to distribute binaries instead of committing them to the repository. A new release will be created by the CI workflow when a tag is pushed to the repository.

Use Github releases to publish binaries instead of committing them.
Remove binaries from the repository, as they are being distributed using
releases instead.
Fails the CI build if no files are found to release.
Tar doesn't like it when the input directory changes, and this prevents that from happening.
@samsface
Copy link
Owner

samsface commented Aug 26, 2024

@CoffeeCoder1 I get what you're doing here but the Godot asset library integrates with the git repo itself. So, this PR would prevent the asset library from consuming this plugin as the binaries would be missing. Also, most Godot people are not super git people. They'll just download all the files of the repo and expect it to work.

@CoffeeCoder1
Copy link
Collaborator Author

@samsface

the Godot asset library integrates with the git repo itself. So, this PR would prevent the asset library from consuming this plugin as the binaries would be missing.

I see. I figured there was probably a reason why it was that way, but I just didn't know what it was. I'm thinking a solution could just be to add back the automated commits but also keep the releases, since they are still a pretty good way to organize things in the repo and they make manual installs a bit easier.

The Godot asset library reads from the repo, so this step is necessary for the asset library to work.
@samsface
Copy link
Owner

What's the need for the release publishing? Do you have some other project that can only grab the binaries from a github release feed? @CoffeeCoder1

@CoffeeCoder1
Copy link
Collaborator Author

@samsface I don't really have any specific use, this is more of just an enhancement than anything. It should make downloading earlier releases and manually installing a bit easier, but I'd imagine that anyone who wants to do a manual install probably already knows how to download a git tag (I think one of the main workflows where this would be useful would be a command-line only workflow, which I'd imagine most Godot users aren't using). If you don't want to merge this that's fine, this was more of a proposal than anything.

@CoffeeCoder1 CoffeeCoder1 added the enhancement New feature or request label Aug 29, 2024
@CoffeeCoder1
Copy link
Collaborator Author

@samsface I just got around to creating an Asset Library account just to see what options are available, and it looks like we may actually be able to point it at a release URL by just selecting the Custom repository host and linking to the ZIP archive for the release. I will need to modify my thing a bit if we want to do this, since my thing only produces .tar.gz archives, but that should be pretty easy.

@CoffeeCoder1 CoffeeCoder1 marked this pull request as draft September 15, 2024 22:10
@CoffeeCoder1 CoffeeCoder1 added the ci Improvements related to CI label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Improvements related to CI enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants