Open CoffeeCoder1 opened 2 months ago
@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.
@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.
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
@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.
@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.
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.