Skip to content

Fixing issue with slack notification#232

Merged
nr-rkallempudi merged 1 commit intomainfrom
NR-517619-rkallempudi
Feb 16, 2026
Merged

Fixing issue with slack notification#232
nr-rkallempudi merged 1 commit intomainfrom
NR-517619-rkallempudi

Conversation

@nr-rkallempudi
Copy link
Copy Markdown
Contributor

Summary

Fixes GitHub Actions output format error in the merge_to_main.yml workflow that was preventing Slack notifications from being sent when releases are published.

Problem

The notify job was failing with:

Error: Unable to process file command 'output' successfully.
Error: Invalid format 'fb-windows-4.2.2-amd64.zip'

This occurred because:

  1. Multiline output issue: The artifacts list was using join("\n") to create newline-separated values, but was using the old echo "artifacts=$ARTIFACTS" format which doesn't support multiline values in GitHub Actions
  2. Invalid JSON in Slack payload: Even if the multiline output worked, embedding newlines directly into JSON string values is invalid and would cause the Slack webhook to fail

Solution

Changed the artifact list format from newline-separated to comma-separated:

  • Modified join("\n") to join(", ") in the jq command
  • This creates a single-line output that:
    • ✅ Works with the simple echo "artifacts=$ARTIFACTS" format
    • ✅ Generates valid JSON for the Slack payload
    • ✅ Displays cleanly in Slack notifications as a comma-separated list

@nr-rkallempudi nr-rkallempudi changed the title fixing issue with slack notification Fixing issue with slack notification Feb 16, 2026
@nr-rkallempudi
Copy link
Copy Markdown
Contributor Author

@copilot

Copy link
Copy Markdown

Copilot AI commented Feb 16, 2026

@nr-rkallempudi I've opened a new pull request, #233, to work on those changes. Once the pull request is ready, I'll request review from you.

@nr-rkallempudi nr-rkallempudi merged commit 01dad33 into main Feb 16, 2026
12 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.

3 participants