Skip to content

Commit e929f40

Browse files
committed
workflows: improve generate dockerfile workflow
- Run under perl:devel container - Run only for changes on selected paths - Install prereqs via cpm - Report possible uncommitted changes back to PR
1 parent 378aed9 commit e929f40

File tree

1 file changed

+65
-12
lines changed

1 file changed

+65
-12
lines changed

.github/workflows/generate-dockerfiles-patches.yml

Lines changed: 65 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,32 +3,85 @@ name: Generate Dockerfiles/patches
33
on:
44
push:
55
branches:
6-
- '**'
6+
- master
77
tags-ignore:
88
- '*'
99
pull_request:
10+
paths:
11+
- cpanfile
12+
- config.yml
13+
- generate.pl
14+
- .github/workflows/generate-dockerfiles-patches.yml
1015

1116
jobs:
1217
generate:
1318
runs-on: ubuntu-latest
19+
container: perl:devel
20+
permissions:
21+
contents: read
22+
pull-requests: write
23+
1424
steps:
25+
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
26+
with:
27+
fetch-depth: 0
28+
1529
- name: Set up git user name and email
1630
run: |
1731
git config --global user.email "test@github-actions"
1832
git config --global user.name "GitHub Actions"
19-
- uses: actions/checkout@master
20-
- name: Install system perl and cpanm
21-
run: |
22-
sudo apt-get install --no-install-recommends -y perl cpanminus
33+
2334
- name: Install dependencies
2435
run: |
25-
cpanm --quiet --installdeps --notest -L local .
36+
cpm install -g
37+
2638
- name: Generate Dockerfiles/patches
39+
id: generate
40+
shell: bash -euo pipefail -x {0}
2741
run: |
28-
perl -Ilocal/lib/perl5 ./generate.pl
29-
- name: Show diffstat (if any)
30-
run: |
31-
git --no-pager diff --stat HEAD
32-
- name: Show diffstat (if any)
42+
export DOCKER_PERL_DOWNLOADS_DIR=/tmp/docker-perl-downloads
43+
perl ./generate.pl
44+
/usr/bin/git --no-pager diff --stat > diffstat.txt
45+
if [[ -s diffstat.txt ]]; then
46+
echo has_extra_diffs=1 >> $GITHUB_OUTPUT
47+
fi
48+
49+
- name: Setup NodeJS in Perl container (for comment to PR)
50+
if: github.event_name == 'pull_request' && steps.generate.outputs.has_extra_diffs
3351
run: |
34-
git --no-pager diff --stat HEAD
52+
if ! command -v node; then
53+
curl -sL https://deb.nodesource.com/setup_lts.x -o nodesource_setup.sh
54+
bash nodesource_setup.sh
55+
apt-get install -y --no-install-recommends nodejs
56+
npm install -g yarn
57+
rm -fr /var/lib/apt/lists/* /var/lib/apt/cache/* nodesource_setup.sh
58+
fi
59+
60+
- name: Comment diffstat of generated changes to PR (if any)
61+
if: github.event_name == 'pull_request' && steps.generate.outputs.has_extra_diffs
62+
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
63+
with:
64+
script: |
65+
const fs = require('node:fs')
66+
let out = fs.readFileSync('diffstat.txt', 'utf8').trim()
67+
68+
if (out && context.actor !== 'nektos/act') {
69+
const output = `
70+
#### :warning: Additional changes generated (missing in commit), see diffstat:
71+
<details><summary>Show Output</summary>
72+
73+
\`\`\`diff
74+
${ out }
75+
\`\`\`
76+
</details>`
77+
78+
github.rest.issues.createComment({
79+
issue_number: context.issue.number,
80+
owner: context.repo.owner,
81+
repo: context.repo.repo,
82+
body: output
83+
})
84+
} else {
85+
console.log(out)
86+
}
87+

0 commit comments

Comments
 (0)