Skip to content

Commit

Permalink
[nrf noup] Fix parsing PR description
Browse files Browse the repository at this point in the history
Use `env` instead of assigning PR body to variable, as `` ` `` is
interpreted as shell command substitution.

Signed-off-by: Adrian Gielniewski <[email protected]>
  • Loading branch information
adigie authored and ArekBalysNordic committed Feb 27, 2025
1 parent ded174b commit 846c831
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/examples-nrfconnect.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,9 @@ jobs:
echo "${{ github.event.inputs.ncs_revision }}" > config/nrfconnect/.nrfconnect-recommended-revision
- name: Update NCS recommended version (pull_request)
if: github.event_name == 'pull_request'
env:
PR_DESCRIPTION: "${{ github.event.pull_request.body }}"
run: |
PR_DESCRIPTION="${{ github.event.pull_request.body }}"
if [ -n "$PR_DESCRIPTION" ]; then
NCS_PR_ID=$(echo $PR_DESCRIPTION | { grep 'NCS PR' || true; } | sed 's/.*nrfconnect\/sdk-nrf//' | { grep -oE '[0-9]+' || true; })
if [ -n "$NCS_PR_ID" ]; then
Expand Down

0 comments on commit 846c831

Please sign in to comment.