-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update module github-release action #926
Comments
It would also be great if the release getting cut could run directly in CI vs. the current process of cloning the main repo and running the release task locally. |
That seems like a good idea to me: 👍 |
I don’t understand what you want to say with this. What’s wrong with this in your opinion? Can you please go more into detail? I’m happy to change this, but I simply don’t understand it 😅🤔 |
You mean wrong with this proposal, or wrong with the current process? As I understand it, the intent here is to create a release PR, correct? Didn't mean to take this in an OT direction. What I'm saying (just commenting via a thread in IRC) is that it would also be nice at some point if it were possible to somehow have a workflow in GitHub Actions could run the Probably some technical details that may make it hard to do safely / securely, so appreciate it may not be possible, just kind of a "it would be nice" type of aside. |
I think our 'release process' are three steps right now:
It's possible to enhance all of those steps. Why I created this issue: To improve step 3 and attach the .tar.gz to the github release. Step 1 and 2 currently run locallly. And I would like to have an option to run them locally or in CI. And I think @wyardley had the same idea. |
ah okay, got you totally wrong.... my intend and the thing initially mentioned in the first post here in the issue was only to generate the gh release page, nothing else.... but doing more and having a more complete and also streamlined release process is totally understandable. sry for the confusion :D |
Yeah, sorry, I made an aside about it on IRC, and Tim just mentioned maybe commenting on it here; sorry for the lacking context in my earlier comment. Side note: there is some prior art for tools that automate release creation (tools like release-please, but most of them use commit messages vs. labels as the source of changelog information. I personally have had good experiences with semantic-release and its ilk, but also understand Vox's reasons for doing things the way we do with labels. Also, if it's at all helpful for any of this, also played around recently with using a GitHub app and, e.g., |
A quick update here: The source is at: https://github.com/voxpupuli/gha-create-a-github-release/blob/v1/action.yml This creates github releases, like this: https://github.com/voxpupuli/puppet-bolt/releases/tag/v1.6.0 it probably makes sense to run those two actions in a single job. That enables the second action to access the module artifact from the first action. That can be attached to the github release. voxpupuli/gha-puppet#66 is the first step to automatically generate release PRs. |
@rwaffen implemented a nice action that creates github releases: https://github.com/voxpupuli/modulesync_config/blob/master/moduleroot/.github/workflows/release.yml.erb#L35-L40
As you can see here: https://github.com/voxpupuli/puppet-quadlets/releases/tag/v1.0.0
I observed two things:
The text was updated successfully, but these errors were encountered: