Skip to content

Commit cda74ee

Browse files
committed
fixup
Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
1 parent 0b2ab3d commit cda74ee

File tree

1 file changed

+35
-34
lines changed

1 file changed

+35
-34
lines changed

.github/workflows/doc-update.yml

Lines changed: 35 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ jobs:
7878
lintreport: './markdownlint-report.txt'
7979
outputs:
8080
proceed: ${{ steps.report-exists.outputs.proceed }}
81-
congrats: ${{ steps.congrats.output.proceed }}
81+
congrats: ${{ steps.congrats.outputs.proceed }}
8282
report: ${{ steps.report-exists.outputs.report }}
8383

8484
steps:
@@ -149,7 +149,7 @@ jobs:
149149
spellcheckreport: './spellcheck-report.txt'
150150
outputs:
151151
proceed: ${{ steps.report-exists.outputs.proceed }}
152-
congrats: ${{ steps.congrats.output.proceed }}
152+
congrats: ${{ steps.congrats.outputs.proceed }}
153153
report: ${{ steps.report-exists.outputs.report }}
154154
steps:
155155
- uses: actions/checkout@v5
@@ -220,13 +220,13 @@ jobs:
220220
if: ${{ needs.markdown-lint.outputs.congrats == 'true' }}
221221
runs-on: ubuntu-latest
222222
outputs:
223-
run_id: ${{ steps.notify_markdown_congrats.run_id }}
224-
target_repo: ${{ steps.notify_markdown_congrats.target_repo }}
225-
pr_number: ${{ steps.notify_markdown_congrats.pr_number }}
226-
pr_sha: ${{ steps.notify_markdown_congrats.pr_sha }}
227-
artifact_name: ${{ steps.notify_markdown_congrats.artifact_name }}
228-
comment_title: ${{ steps.notify_markdown_congrats.comment_title }}
229-
reactions: ${{ steps.notify_markdown_congrats.reactions }}
223+
run_id: ${{ steps.notify_markdown_congrats.outputs.run_id }}
224+
target_repo: ${{ steps.notify_markdown_congrats.outputs.target_repo }}
225+
pr_number: ${{ steps.notify_markdown_congrats.outputs.pr_number }}
226+
pr_sha: ${{ steps.notify_markdown_congrats.outputs.pr_sha }}
227+
artifact_name: ${{ steps.notify_markdown_congrats.outputs.artifact_name }}
228+
comment_title: ${{ steps.notify_markdown_congrats.outputs.comment_title }}
229+
reactions: ${{ steps.notify_markdown_congrats.outputs.reactions }}
230230
steps:
231231
- name: Congrats
232232
id: notify_markdown_congrats
@@ -238,7 +238,8 @@ jobs:
238238
Markdown looks good to me. 👍
239239
EOF
240240
241-
echo "${MSG}" > "${{ env.artifacts_dir}}/${{ env.markdown_artifact }}"
241+
export MSG
242+
printenv MSG > "${{ env.artifacts_dir}}/${{ env.markdown_artifact }}"
242243
243244
echo "### Your changes to markdown docs look good. 👍" >> $GITHUB_STEP_SUMMARY
244245
echo "" >> $GITHUB_STEP_SUMMARY
@@ -276,13 +277,13 @@ jobs:
276277
needs: markdown-lint
277278
if: ${{ needs.markdown-lint.outputs.proceed == 'true' }}
278279
outputs:
279-
run_id: ${{ steps.notify_markdown_report.run_id }}
280-
target_repo: ${{ steps.notify_markdown_report.target_repo }}
281-
pr_number: ${{ steps.notify_markdown_report.pr_number }}
282-
pr_sha: ${{ steps.notify_markdown_report.pr_sha }}
283-
artifact_name: ${{ steps.notify_markdown_report.artifact_name }}
284-
comment_title: ${{ steps.notify_markdown_report.comment_title }}
285-
reactions: ${{ steps.notify_markdown_report.reactions }}
280+
run_id: ${{ steps.notify_markdown_report.outputs.run_id }}
281+
target_repo: ${{ steps.notify_markdown_report.outputs.target_repo }}
282+
pr_number: ${{ steps.notify_markdown_report.outputs.pr_number }}
283+
pr_sha: ${{ steps.notify_markdown_report.outputs.pr_sha }}
284+
artifact_name: ${{ steps.notify_markdown_report.outputs.artifact_name }}
285+
comment_title: ${{ steps.notify_markdown_report.outputs.comment_title }}
286+
reactions: ${{ steps.notify_markdown_report.outputs.reactions }}
286287
runs-on: ubuntu-latest
287288
steps:
288289
- name: Format PR comment
@@ -354,13 +355,13 @@ jobs:
354355
needs: markdown-spelling
355356
if: ${{ needs.markdown-spelling.outputs.congrats == 'true' }}
356357
outputs:
357-
run_id: ${{ steps.notify_spelling_congrats.run_id }}
358-
target_repo: ${{ steps.notify_spelling_congrats.target_repo }}
359-
pr_number: ${{ steps.notify_spelling_congrats.pr_number }}
360-
pr_sha: ${{ steps.notify_spelling_congrats.pr_sha }}
361-
artifact_name: ${{ steps.notify_spelling_congrats.artifact_name }}
362-
comment_title: ${{ steps.notify_spelling_congrats.comment_title }}
363-
reactions: ${{ steps.notify_spelling_congrats.reactions }}
358+
run_id: ${{ steps.notify_spelling_congrats.outputs.run_id }}
359+
target_repo: ${{ steps.notify_spelling_congrats.outputs.target_repo }}
360+
pr_number: ${{ steps.notify_spelling_congrats.outputs.pr_number }}
361+
pr_sha: ${{ steps.notify_spelling_congrats.outputs.pr_sha }}
362+
artifact_name: ${{ steps.notify_spelling_congrats.outputs.artifact_name }}
363+
comment_title: ${{ steps.notify_spelling_congrats.outputs.comment_title }}
364+
reactions: ${{ steps.notify_spelling_congrats.outputs.reactions }}
364365
runs-on: ubuntu-latest
365366
steps:
366367
- name: Congrats
@@ -373,7 +374,8 @@ jobs:
373374
Spelling in mardown looks good to me. 👍
374375
EOF
375376
376-
echo "${MSG}" > "${{ env.artifacts_dir }}/${{ env.spellcheck_artifact }}"
377+
export MSG
378+
printenv MSG > "${{ env.artifacts_dir }}/${{ env.spellcheck_artifact }}"
377379
378380
echo "run_id=${{ github.run_id }}" >> "$GITHUB_OUTPUT"
379381
echo "target_repo=${{ github.repository }}" >> "$GITHUB_OUTPUT"
@@ -407,13 +409,13 @@ jobs:
407409
needs: markdown-spelling
408410
if: ${{ needs.markdown-spelling.outputs.proceed == 'true' }}
409411
outputs:
410-
run_id: ${{ steps.notify_spelling_report.run_id }}
411-
target_repo: ${{ steps.notify_spelling_report.target_repo }}
412-
pr_number: ${{ steps.notify_spelling_report.pr_number }}
413-
pr_sha: ${{ steps.notify_spelling_report.pr_sha }}
414-
artifact_name: ${{ steps.notify_spelling_report.artifact_name }}
415-
comment_title: ${{ steps.notify_spelling_report.comment_title }}
416-
reactions: ${{ steps.notify_spelling_report.reactions }}
412+
run_id: ${{ steps.notify_spelling_report.outputs.run_id }}
413+
target_repo: ${{ steps.notify_spelling_report.outputs.target_repo }}
414+
pr_number: ${{ steps.notify_spelling_report.outputs.pr_number }}
415+
pr_sha: ${{ steps.notify_spelling_report.outputs.pr_sha }}
416+
artifact_name: ${{ steps.notify_spelling_report.outputs.artifact_name }}
417+
comment_title: ${{ steps.notify_spelling_report.outputs.comment_title }}
418+
reactions: ${{ steps.notify_spelling_report.outputs.reactions }}
417419
runs-on: ubuntu-latest
418420
steps:
419421
- name: Format PR comment
@@ -438,7 +440,6 @@ jobs:
438440
- name: Slap on the wrist
439441
env:
440442
OUTPUT: "${{ steps.comment_formatter.outputs.updated-text }}"
441-
id: notify_spelling_repo
442443
run: |
443444
mkdir -p "${{ env.artifacts_dir }}"
444445
printenv OUTPUT > "${{ env.artifacts_dir}}/${{ env.spellcheck_artifact }}"
@@ -457,7 +458,7 @@ jobs:
457458
name: ${{ env.spellcheck_artifact }}
458459

459460
- name: Notify
460-
id: spelling_report_notify
461+
id: notify_spelling_report
461462
run: |
462463
echo "::notice::Commented pull request ${{ github.event.pull_request.number }}"
463464

0 commit comments

Comments
 (0)