Skip to content

fix(ci): handle multiline commit messages in notifications - #15

Merged
0xJacky merged 1 commit into
mainfrom
codex/fix-notify-multiline-message
Sep 1, 2026
Merged

0xJacky merged 1 commit into
mainfrom
codex/fix-notify-multiline-message

Conversation

@0xJacky

@0xJacky 0xJacky commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Avoid writing the squash commit message directly to GITHUB_ENV, where multiline Co-authored-by trailers are parsed as invalid environment-file records. Read the message directly from the workflow_run context in the notification payload instead, and use the correct workflow_run pull_requests context for PR authors.

@0xJacky
0xJacky merged commit 3aa322b into main Sep 1, 2026
2 checks passed
@0xJacky
0xJacky deleted the codex/fix-notify-multiline-message branch September 1, 2026 04:38
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.

1 participant