Skip to content

Conversation

@x0rw
Copy link
Contributor

@x0rw x0rw commented Jul 31, 2025

This PR fixes an issue with improper quote escaping in the CI workflow that caused unexpected parsing behavior when passing the GitHub event issue payload to the Python script.
Before:

  cat <<EOF | uv run python scripts/auto-pr-helper.py --save
  ${{ toJson(github.event.issue) }}
  EOF

This version failed when the payload contained special characters or quotes.

After:

cat <<'EOF' | uv run python scripts/auto-pr-helper.py --save
${{ toJson(github.event.issue) }}
EOF

By quoting the here-document delimiter with single quotes (<<'EOF'), we ensure the payload is passed literally to the script, preserving all quote characters safely.

Origin:

This was noticed in PR #159, it was missing the code section.

https://github.com/x0rw/safety-critical-rust-coding-guidelines/actions/runs/16647842314/job/47112691866

fixes #161

@netlify
Copy link

netlify bot commented Jul 31, 2025

Deploy Preview for scrc-coding-guidelines ready!

Name Link
🔨 Latest commit 62c93df
🔍 Latest deploy log https://app.netlify.com/projects/scrc-coding-guidelines/deploys/688b881bc72468000845a2ae
😎 Deploy Preview https://deploy-preview-160--scrc-coding-guidelines.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@PLeVasseur PLeVasseur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix @x0rw!

@PLeVasseur PLeVasseur added this pull request to the merge queue Aug 1, 2025
Merged via the queue into rustfoundation:main with commit 444a4e5 Aug 1, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate fix for issue => PR action

2 participants