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 release prep script #243

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

mhashizume
Copy link
Contributor

This commit adds the release script prep necessary for calling the auto_release_prep workflow.

This script is taken from https://github.com/puppetlabs/release-engineering-repo-standards

@mhashizume mhashizume requested a review from a team as a code owner February 13, 2024 20:57
@@ -0,0 +1,5 @@
#!/usr/bin/env bash

docker run -it --rm -e CHANGELOG_GITHUB_TOKEN -v $(pwd):/usr/local/src/your-app \
Copy link
Contributor

Choose a reason for hiding this comment

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

your-app seems a little bit suspect here, is it supposed to be replaced with maybe beaker-puppet?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think that's okay--from a quick look at other repos that use this automation, it seems like that path is local to the container and more or less a temporary staging area for the changelog after it's been generated.

Tefoji ran a release prep action a few weeks ago: https://github.com/puppetlabs/tefoji/actions/runs/7732718060/job/21083254959

Which shows this in the logs:

 Generated log placed in /usr/local/src/your-app/CHANGELOG.md

And created this PR: https://github.com/puppetlabs/tefoji/pull/107

This commit adds the release script prep necessary for calling the
auto_release_prep workflow.

This script is taken from https://github.com/puppetlabs/release-engineering-repo-standards
@joshcooper joshcooper merged commit b78c7d1 into puppetlabs:master Feb 14, 2024
6 checks passed
@joshcooper
Copy link
Contributor

Sorry meant to just approve and let Tony review, but I got carried away

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.

3 participants