Skip to content

Commit 1d15a60

Browse files
committed
style: Use single quotes
1 parent 82d00dd commit 1d15a60

File tree

1 file changed

+17
-19
lines changed

1 file changed

+17
-19
lines changed

.github/workflows/ci.yaml

+17-19
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: CI
22

33
on:
44
schedule:
5-
- cron: '30 9 * * *' # Pacific Time 01:30 AM in UTC
5+
- cron: '30 9 * * *' # Pacific Time 01:30 AM in UTC
66
pull_request:
77
types:
88
- opened
@@ -25,7 +25,7 @@ on:
2525
required: false
2626
MERGE_BUMPED_MANIFEST:
2727
type: boolean
28-
description: "(used if BUMP_MANIFEST=true) If true: attempt to PR/merge manifest branch"
28+
description: '(used if BUMP_MANIFEST=true) If true: attempt to PR/merge manifest branch'
2929
default: false
3030
required: false
3131

@@ -34,16 +34,15 @@ concurrency:
3434
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
3535

3636
permissions:
37-
contents: write # to fetch code and push branch
38-
actions: write # to cancel previous workflows
39-
packages: write # to upload container
40-
pull-requests: write # to make pull request for manifest bump
37+
contents: write # to fetch code and push branch
38+
actions: write # to cancel previous workflows
39+
packages: write # to upload container
40+
pull-requests: write # to make pull request for manifest bump
4141

4242
env:
4343
DEFAULT_MANIFEST_ARTIFACT_NAME: bumped-manifest
4444

4545
jobs:
46-
4746
metadata:
4847
runs-on: ubuntu-22.04
4948
outputs:
@@ -115,7 +114,7 @@ jobs:
115114
shell: bash -x -e {0}
116115
run: |
117116
bash bump.sh --input-manifest manifest.yaml --output-manifest manifest.yaml.new --base-patch-dir ./patches-new
118-
117+
119118
- name: Maybe replace current manifest/patches with the new one and show diff
120119
working-directory: .github/container
121120
shell: bash -x -e {0}
@@ -168,12 +167,11 @@ jobs:
168167
steps:
169168
- name: "Tests Succeeded: ${{ !contains(needs.*.result, 'failure') && !contains(needs.*.result, 'cancelled') }}"
170169
id: test_result
171-
run:
172-
echo "SUCCEEDED=${{ !contains(needs.*.result, 'failure') && !contains(needs.*.result, 'cancelled') }}" | tee -a $GITHUB_OUTPUT
170+
run: echo "SUCCEEDED=${{ !contains(needs.*.result, 'failure') && !contains(needs.*.result, 'cancelled') }}" | tee -a $GITHUB_OUTPUT
173171

174172
- name: Check out the repository under ${GITHUB_WORKSPACE}
175173
uses: actions/checkout@v4
176-
174+
177175
- name: Delete checked-out manifest and patches
178176
run: |
179177
rm .github/container/manifest.yaml
@@ -185,7 +183,7 @@ jobs:
185183
name: ${{ needs.metadata.outputs.MANIFEST_ARTIFACT_NAME }}
186184
path: .github/container/
187185

188-
- name: "Create local manifest branch: ${{ needs.metadata.outputs.MANIFEST_BRANCH }}"
186+
- name: 'Create local manifest branch: ${{ needs.metadata.outputs.MANIFEST_BRANCH }}'
189187
id: local_branch
190188
shell: bash -x -e {0}
191189
run: |
@@ -213,7 +211,7 @@ jobs:
213211
git merge --ff-only ${{ needs.metadata.outputs.MANIFEST_BRANCH }}
214212
# Push the new change
215213
git push origin ${{ github.ref_name }}
216-
214+
217215
# We will create a Draft PR & remote branch if:
218216
# 1. The tests failed
219217
# 2. The merge failed
@@ -244,12 +242,12 @@ jobs:
244242
draft: true
245243
env:
246244
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
247-
248-
- name: "Log created PR: #${{ fromJson(steps.create_pr.outputs.data).number }}"
245+
246+
- name: 'Log created PR: #${{ fromJson(steps.create_pr.outputs.data).number }}'
249247
if: steps.create_pr.outcome == 'success'
250248
run: |
251249
echo "https://github.com/NVIDIA/JAX-Toolbox/pull/${{ fromJson(steps.create_pr.outputs.data).number }}" | tee -a $GITHUB_STEP_SUMMARY
252-
250+
253251
# Guard delete in simple check to protect other branches
254252
- name: Check that the branch matches znightly- prefix
255253
run: |
@@ -271,7 +269,7 @@ jobs:
271269

272270
make-publish-configs:
273271
runs-on: ubuntu-22.04
274-
if: ${{ !cancelled() }}
272+
if: ${{ !cancelled() }}
275273
env:
276274
MEALKIT_IMAGE_REPO: ${{ needs.metadata.outputs.PUBLISH == 'true' && 'jax-mealkit' || 'mock-jax-mealkit' }}
277275
FINAL_IMAGE_REPO: ${{ needs.metadata.outputs.PUBLISH == 'true' && 'jax' || 'mock-jax' }}
@@ -365,7 +363,7 @@ jobs:
365363
needs:
366364
- metadata
367365
- make-publish-configs
368-
if: ${{ !cancelled() && needs.make-publish-configs.outputs.PUBLISH_CONFIGS.config != '{"config":[]}' }}
366+
if: ${{ !cancelled() && needs.make-publish-configs.outputs.PUBLISH_CONFIGS.config != '{"config":[]}' }}
369367
strategy:
370368
fail-fast: false
371369
matrix: ${{ fromJson(needs.make-publish-configs.outputs.PUBLISH_CONFIGS) }}
@@ -381,7 +379,7 @@ jobs:
381379
382380
finalize:
383381
needs: [metadata, amd64, arm64, publish-containers]
384-
if: "!cancelled()"
382+
if: '!cancelled()'
385383
uses: ./.github/workflows/_finalize.yaml
386384
with:
387385
BUILD_DATE: ${{ needs.metadata.outputs.BUILD_DATE }}

0 commit comments

Comments
 (0)