Skip to content

Commit 7131ed7

Browse files
committed
Fixes to license GHA and version updates.
Signed-off-by: David Venable <dlv@amazon.com>
1 parent 5d2fa84 commit 7131ed7

1 file changed

Lines changed: 18 additions & 14 deletions

File tree

.github/workflows/license-header-check.yml

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,12 @@ jobs:
2020

2121
steps:
2222
- name: Checkout code
23-
uses: actions/checkout@v4
23+
uses: actions/checkout@v6
2424
with:
2525
fetch-depth: 0
2626

2727
- name: Setup Python
28-
uses: actions/setup-python@v4
28+
uses: actions/setup-python@v6
2929
with:
3030
python-version: '3.9'
3131

@@ -36,20 +36,22 @@ jobs:
3636
3737
- name: Comment on PR
3838
if: failure() && steps.license-check.outputs.violations != ''
39-
uses: actions/github-script@v7
39+
uses: actions/github-script@v8
4040
with:
4141
script: |
42-
const violations = `${{ steps.license-check.outputs.violations }}`;
42+
const violations = process.env.VIOLATIONS;
4343
44-
const body = `## ⚠️ License Header Violations Found
45-
46-
The following newly added files are missing required license headers:
47-
48-
${violations}
49-
50-
Please add the appropriate license header to each file and push your changes.
51-
52-
**See the license header requirements:** https://github.com/opensearch-project/data-prepper/blob/main/CONTRIBUTING.md#license-headers`;
44+
const body = [
45+
'## ⚠️ License Header Violations Found',
46+
'',
47+
'The following newly added files are missing required license headers:',
48+
'',
49+
violations,
50+
'',
51+
'Please add the appropriate license header to each file and push your changes.',
52+
'',
53+
'**See the license header requirements:** https://github.com/opensearch-project/data-prepper/blob/main/CONTRIBUTING.md#license-headers'
54+
].join('\n');
5355
5456
const { data: comments } = await github.rest.issues.listComments({
5557
owner: context.repo.owner,
@@ -76,4 +78,6 @@ jobs:
7678
issue_number: context.issue.number,
7779
body: body
7880
});
79-
}
81+
}
82+
env:
83+
VIOLATIONS: ${{ steps.license-check.outputs.violations }}

0 commit comments

Comments
 (0)