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 action to create a module release PR #66

Open
wants to merge 1 commit into
base: v3
Choose a base branch
from
Open

Add action to create a module release PR #66

wants to merge 1 commit into from

Conversation

bastelfreak
Copy link
Member

No description provided.

.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the prep branch 2 times, most recently from eb2e843 to 672a436 Compare December 19, 2024 16:29
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the prep branch 2 times, most recently from db2ad4d to 09a0cb5 Compare December 19, 2024 17:01
@ekohl ekohl added the enhancement New feature or request label Dec 19, 2024
@bastelfreak bastelfreak force-pushed the prep branch 2 times, most recently from 64eaaf5 to 0d0c225 Compare December 21, 2024 23:28
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the prep branch 2 times, most recently from 2114a18 to 467cb1f Compare December 22, 2024 00:20
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
.github/workflows/prepare_release.yml Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the prep branch 5 times, most recently from 166c090 to e3c3bba Compare December 22, 2024 11:30
Co-authored-by: Ewoud Kohl van Wijngaarden <[email protected]>
steps:
- name: Checkout repository
uses: actions/checkout@v4
with:
Copy link
Member Author

Choose a reason for hiding this comment

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

Perforce makes a full clone with:

      - name: "Checkout"
        uses: "actions/checkout@v4"
        with:
          fetch-depth: 0

In my tests that wasn't required. Maybe it's for some edge cases. Based on their git history I didn't find any explanation.

@bastelfreak bastelfreak marked this pull request as ready for review December 22, 2024 11:35
@bastelfreak
Copy link
Member Author

bastelfreak commented Dec 22, 2024

Tested successfully at voxpupuli/puppet-example#82 . Followup PR for the optional input: voxpupuli/puppet-example#83

# https://docs.github.com/en/actions/security-for-github-actions/security-guides/automatic-token-authentication
CHANGELOG_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: bundle exec rake release:prepare
- name: Create pull Request
Copy link
Member

Choose a reason for hiding this comment

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

Looking at voxpupuli/puppet-example#82 I wonder if we can come up with some better instructions in the body. For example, it could link to https://voxpupuli.org/docs/releasing_version/ though that isn't really explaining it well for this new process.

Copy link
Member Author

Choose a reason for hiding this comment

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

My idea was to link to https://voxpupuli.org/docs/releasing_version/ and update it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants