Skip to content

Commit becb38c

Browse files
authored
chore(ci): add linter for GitHub Actions as pre-commit hook (aws-powertools#1479)
1 parent 1b85b0b commit becb38c

6 files changed

+12
-5
lines changed

.github/workflows/build_changelog.yml

-1
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,4 @@ on:
66

77
jobs:
88
changelog:
9-
needs: release
109
uses: ./.github/workflows/reusable_publish_changelog.yml

.github/workflows/on_opened_pr.yml

-2
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@ jobs:
2020
runs-on: ubuntu-latest
2121
steps:
2222
- uses: actions/checkout@v3
23-
- name: "Debug workflow_run event"
24-
run: echo "${{ github }}"
2523
- name: "Ensure related issue is present"
2624
uses: actions/github-script@v6
2725
env:

.github/workflows/python_build.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ jobs:
2828
matrix:
2929
python-version: [3.7, 3.8, 3.9]
3030
env:
31-
OS: ${{ matrix.os }}
3231
PYTHON: ${{ matrix.python-version }}
3332
steps:
3433
- uses: actions/checkout@v3
@@ -56,6 +55,6 @@ jobs:
5655
with:
5756
file: ./coverage.xml
5857
# flags: unittests
59-
env_vars: OS,PYTHON
58+
env_vars: PYTHON
6059
name: aws-lambda-powertools-python-codecov
6160
# fail_ci_if_error: true # failing more consistently making CI unreliable despite all tests above passing

.github/workflows/reusable_deploy_layer_stack.yml

+3
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,15 @@ on:
88
workflow_call:
99
inputs:
1010
stage:
11+
description: "Deployment stage (BETA, PROD)"
1112
required: true
1213
type: string
1314
artefact-name:
15+
description: "CDK Layer Artefact name to download"
1416
required: true
1517
type: string
1618
environment:
19+
description: "GitHub Environment to use for encrypted secrets"
1720
required: true
1821
type: string
1922

.github/workflows/reusable_export_pr_details.yml

+3
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,16 @@ on:
44
workflow_call:
55
inputs:
66
record_pr_workflow_id:
7+
description: "Record PR workflow execution ID to download PR details"
78
required: true
89
type: number
910
workflow_origin: # see https://github.com/awslabs/aws-lambda-powertools-python/issues/1349
11+
description: "Repository full name for runner integrity"
1012
required: true
1113
type: string
1214
secrets:
1315
token:
16+
description: "GitHub Actions temporary and scoped token"
1417
required: true
1518
# Map the workflow outputs to job outputs
1619
outputs:

.pre-commit-config.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -39,3 +39,8 @@ repos:
3939
hooks:
4040
- id: cfn-python-lint
4141
files: examples/.*\.(yaml|yml)$
42+
- repo: https://github.com/rhysd/actionlint
43+
rev: v1.6.16
44+
hooks:
45+
- id: actionlint-docker
46+
args: [-pyflakes=]

0 commit comments

Comments
 (0)