diff options
author | Michael Hashizume <mhashizume@users.noreply.github.com> | 2023-10-04 10:26:35 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-10-04 10:26:35 -0700 |
commit | 907d28ca583e9d38318f7a79f9d1904b979e13f3 (patch) | |
tree | 711e0ec614f102e4df49ef136db80c3117c629fc | |
parent | e18d8b0c3e1099c923d695c7a723fcde1c1a18dd (diff) | |
parent | 0da634638db709bec592e4304bb73bcb2af9d354 (diff) | |
download | puppet-hosts_core-907d28ca583e9d38318f7a79f9d1904b979e13f3.tar.gz puppet-hosts_core-907d28ca583e9d38318f7a79f9d1904b979e13f3.tar.bz2 |
Merge pull request #42 from mhashizume/PA-5804/main/checkout-action
(PA-5804) Update Checkout GitHub Action
-rw-r--r-- | .github/workflows/auto_release.yml | 2 | ||||
-rw-r--r-- | .github/workflows/release.yml | 4 | ||||
-rw-r--r-- | .github/workflows/static_code_analysis.yaml | 2 | ||||
-rw-r--r-- | rakelib/commits.rake | 13 |
4 files changed, 10 insertions, 11 deletions
diff --git a/.github/workflows/auto_release.yml b/.github/workflows/auto_release.yml index 3799f04..5586fd4 100644 --- a/.github/workflows/auto_release.yml +++ b/.github/workflows/auto_release.yml @@ -28,7 +28,7 @@ jobs: echo STEP_START=$(date +%s) >> $GITHUB_ENV - name: "Checkout Source" if: ${{ github.repository_owner == 'puppetlabs' }} - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: fetch-depth: 0 persist-credentials: false diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index d9604ad..0240b29 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -9,7 +9,7 @@ jobs: runs-on: ubuntu-20.04 steps: - name: Checkout code - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: ref: ${{ github.ref }} clean: true @@ -33,7 +33,7 @@ jobs: runs-on: ubuntu-20.04 steps: - name: Checkout code - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: ref: ${{ github.ref }} clean: true diff --git a/.github/workflows/static_code_analysis.yaml b/.github/workflows/static_code_analysis.yaml index b639220..575cde7 100644 --- a/.github/workflows/static_code_analysis.yaml +++ b/.github/workflows/static_code_analysis.yaml @@ -18,7 +18,7 @@ jobs: runs-on: 'ubuntu-20.04' steps: - name: Checkout current PR code - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: fetch-depth: 0 diff --git a/rakelib/commits.rake b/rakelib/commits.rake index 5e0578e..7cee9fa 100644 --- a/rakelib/commits.rake +++ b/rakelib/commits.rake @@ -1,4 +1,4 @@ -desc "verify that commit messages match CONTRIBUTING.md requirements" +desc "verify that commit summaries are properly formatted" task(:commits) do # This rake task looks at the summary from every commit from this branch not # in the branch targeted for a PR. @@ -7,16 +7,15 @@ task(:commits) do %x{git log --no-merges --pretty=%s #{commit_range}}.each_line do |commit_summary| # This regex tests for the currently supported commit summary tokens. # The exception tries to explain it in more full. - if /^Release prep|\((maint|packaging|doc|docs|modules-\d+)\)|revert/i.match(commit_summary).nil? - raise "\n\n\n\tThis commit summary didn't match CONTRIBUTING.md guidelines:\n" \ - "\n\t\t#{commit_summary}\n" \ - "\tThe commit summary (i.e. the first line of the commit message) should start with one of:\n" \ - "\t\t(MODULES-<digits>) # this is most common and should be a ticket at tickets.puppet.com\n" \ + if /^Release prep|\((maint|packaging|doc|docs|modules|pa-\d+)\)|revert/i.match(commit_summary).nil? + raise "\n\n\n\tPlease make sure that your commit summary (i.e. the first line of the commit message) starts with one of the following:\n" \ + "\t\t(PA-<digits>)\n" \ + "\t\t(MODULES-<digits>)\n" \ "\t\t(docs)\n" \ "\t\t(docs)(DOCUMENT-<digits>)\n" \ "\t\t(packaging)\n" "\t\t(maint)\n" \ - "\t\tRelease prep v <tag>\n" \ + "\t\tRelease prep v<tag>\n" \ "\n\tThis test for the commit summary is case-insensitive.\n\n\n" else puts "#{commit_summary}" |