Skip to content

Commit

Permalink
Extract msg from env var to defining in run
Browse files Browse the repository at this point in the history
  • Loading branch information
pitag-ha committed Nov 15, 2023
1 parent 5cb065c commit 309d7ff
Showing 1 changed file with 26 additions and 12 deletions.
38 changes: 26 additions & 12 deletions .github/workflows/comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,19 +16,33 @@ jobs:
env:
workflow_run: <workflow_run_id>
diff_id: <diff_id>
msg: |
This PR introduces a change in some Merlin query response(s), which has been captured by the fuzzy-test CI. The fuzzy-test diffs of the CI have been approved. Info about the diffs that have been approved:
- Distilled data:
- Diff: https://github.com/pitag-ha/merlin/suites//artifacts/
- Diff 256-sha: {{ steps.category_data_diff_hash.outputs.hash }}
- Data on target branch (`master` at (...)): (...)
- Data on PR merge branch, i.e. merge result of source branch (at (...)) and target branch: (...)
- Full responses:
- Diff: (...)
- Diff 256-sha: my-full-responses-sha
- Data on target branch (`master` at (...)): (...)
- Data on PR merge branch, i.e. merge result of source branch (at (...)) and target branch: (...)
# msg: |
# This PR introduces a change in some Merlin query response(s), which has been captured by the fuzzy-test CI. The fuzzy-test diffs of the CI have been approved. Info about the diffs that have been approved:
# - Distilled data:
# - Diff: https://github.com/pitag-ha/merlin/suites//artifacts/
# - Diff 256-sha: {{ steps.category_data_diff_hash.outputs.hash }}
# - Data on target branch (`master` at (...)): (...)
# - Data on PR merge branch, i.e. merge result of source branch (at (...)) and target branch: (...)
# - Full responses:
# - Diff: (...)
# - Diff 256-sha: my-full-responses-sha
# - Data on target branch (`master` at (...)): (...)
# - Data on PR merge branch, i.e. merge result of source branch (at (...)) and target branch: (...)
run: |
msg= $( cat <<EOF
ONe more: This PR introduces a change in some Merlin query response(s), which has been captured by the fuzzy-test CI. The fuzzy-test diffs of the CI have been approved. Info about the diffs that have been approved:
- Distilled data:
- Diff: https://github.com/pitag-ha/merlin/suites/$workflow_run/artifacts/$diff_id
- Diff 256-sha: {{ steps.category_data_diff_hash.outputs.hash }}
- Data on target branch (\`master\` at (...)): (...)
- Data on PR merge branch, i.e. merge result of source branch (at (...)) and target branch: (...)
- Full responses:
- Diff: (...)
- Diff 256-sha: my-full-responses-sha
- Data on target branch (\`master\` at (...)): (...)
- Data on PR merge branch, i.e. merge result of source branch (at (...)) and target branch: (...)
EOF
)
jq -n --arg msg "$msg" '{ body: $msg }' | tee -a body.json
- name: Write comment on PR
Expand Down

0 comments on commit 309d7ff

Please sign in to comment.