Skip to content

Commit 72e5225

Browse files
committed
Add workflow for linting mixins individually
1 parent c09b64b commit 72e5225

File tree

1 file changed

+44
-5
lines changed

1 file changed

+44
-5
lines changed

.github/workflows/lint-mixins.yml

Lines changed: 44 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
name: "Lint Mixins"
2+
name: Mixin
33

44
permissions:
55
contents: read
@@ -8,11 +8,11 @@ on:
88
# To conserve resources we only run tests against main in PRs
99
pull_request:
1010
branches:
11-
- main
11+
- master
1212

1313
jobs:
1414
check-for-changed-mixins:
15-
name: Check for changed mixins
15+
name: Check for changes
1616
runs-on: ubuntu-latest
1717
outputs:
1818
changed-mixins: ${{ steps.changed-mixins.outputs.all_changed_files }}
@@ -37,8 +37,47 @@ jobs:
3737
dir_names: true
3838
dir_names_exclude_current_dir: true
3939
dir_names_max_depth: 1
40-
files: **-mixin/
40+
files: |
41+
**-mixin/
4142
matrix: true
4243

43-
- name: List all changed mixins:
44+
- name: List all changed mixins
4445
run: echo '${{ steps.changed-mixins.outputs.all_changed_files }}'
46+
47+
lint-mixin:
48+
name: Run Mixtool
49+
runs-on: ubuntu-latest
50+
permissions:
51+
issues: write
52+
contents: write
53+
pull-requests: write
54+
repository-projects: write
55+
timeout-minutes: 15
56+
needs: [check-for-changed-mixins]
57+
strategy:
58+
matrix:
59+
mixin: ${{ fromJSON(needs.check-for-changed-mixins.outputs.changed-mixins) }}
60+
fail-fast: false
61+
steps:
62+
- name: Checkout
63+
uses: actions/checkout@v4
64+
with:
65+
fetch-depth: 0
66+
67+
- name: Setup Go
68+
uses: actions/setup-go@v5
69+
with:
70+
go-version: 1.18
71+
72+
- name: Install CI dependencies
73+
run: make install-ci-deps
74+
75+
- name: Install Mixin dependencies
76+
working-directory: ./${{ matrix.mixin }}
77+
run: jb install
78+
79+
- name: Lint Mixin
80+
working-directory: ./${{ matrix.mixin }}
81+
run: mixtool lint mixin.libsonnet
82+
83+

0 commit comments

Comments
 (0)