Skip to content

Commit 9cd9ebc

Browse files
authored
fix: sql-benchmarks.yml YAML syntax (#2398)
Apparently the `${{` is not allowed in the short-hand array syntax. I switched to the dashed list syntax. I added a yaml lint CI step too.
1 parent 0d8f4f7 commit 9cd9ebc

File tree

5 files changed

+51
-6
lines changed

5 files changed

+51
-6
lines changed

.github/workflows/bench.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ on:
55
branches: [ develop ]
66

77
permissions:
8-
id-token: write # enables AWS-GitHub OIDC
8+
id-token: write # enables AWS-GitHub OIDC
99
actions: read
1010
contents: write
1111
deployments: write

.github/workflows/ci.yml

+15-3
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,26 @@ on:
1313
permissions:
1414
actions: read
1515
contents: read
16-
checks: write # audit-check creates checks
17-
issues: write # audit-check creates issues
16+
checks: write # audit-check creates checks
17+
issues: write # audit-check creates issues
1818

1919
env:
2020
CARGO_TERM_COLOR: always
2121
RUST_BACKTRACE: 1
2222

2323
jobs:
24+
validate-workflow-yaml:
25+
runs-on: ubuntu-latest
26+
steps:
27+
- uses: actions/checkout@v2
28+
- name: Validate YAML file
29+
run: |
30+
# Lint the workflows and yamllint's configuration file.
31+
yamllint \
32+
-c yamllint-config.yaml \
33+
yamllint-config.yaml \
34+
.github/workflows
35+
2436
python-lint:
2537
name: "Python (lint)"
2638
runs-on: ubuntu-latest
@@ -130,7 +142,7 @@ jobs:
130142
uses: taiki-e/install-action@cargo-hack
131143
- name: Rust Build (${{matrix.config.name}})
132144
run: cargo hack ${{matrix.config.command}} --locked ${{matrix.config.args}} --ignore-private
133-
145+
134146
check-min-deps:
135147
name: "Check build with minimal dependencies"
136148
runs-on: ubuntu-latest

.github/workflows/fuzz.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: Fuzz
22

33
on:
44
schedule:
5-
- cron: '0 0 * * *' # daily
5+
- cron: '0 0 * * *' # daily
66
workflow_dispatch:
77

88
jobs:

.github/workflows/sql-pr.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,4 +26,4 @@ jobs:
2626
needs: label_trigger
2727
uses: ./.github/workflows/sql-benchmarks.yml
2828
with:
29-
mode: 'pr'
29+
mode: 'pr'

yamllint-config.yaml

+33
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
---
2+
3+
yaml-files:
4+
- '*.yaml'
5+
- '*.yml'
6+
- '.yamllint'
7+
8+
rules:
9+
anchors: enable
10+
braces: disable
11+
brackets: disable
12+
colons: enable
13+
commas: enable
14+
comments:
15+
level: warning
16+
comments-indentation:
17+
level: warning
18+
document-end: disable
19+
document-start: disable
20+
empty-lines: enable
21+
empty-values: disable
22+
float-values: disable
23+
hyphens: enable
24+
indentation: enable
25+
key-duplicates: enable
26+
key-ordering: disable
27+
line-length: disable
28+
new-line-at-end-of-file: enable
29+
new-lines: enable
30+
octal-values: disable
31+
quoted-strings: disable
32+
trailing-spaces: enable
33+
truthy: disable

0 commit comments

Comments
 (0)