Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 28, 2025

The PR merge notification workflow was only sending basic information to Discord, making it difficult for users to understand what was merged without manually looking up the PR.

Before:

Pull Request 123 merged by testuser

After:

Pull Request #123 "Fix LED brightness issue" merged by testuser
https://github.com/wled/WLED/pull/123

The solution uses environment variables and jq for safe JSON escaping to handle edge cases like PR titles containing quotes, special characters, or newlines. This approach is consistent with existing workflow patterns in the repository and prevents potential JSON injection issues.

Changes:

  • Added PR title in quotes to the notification message
  • Added clickable PR URL on a new line
  • Used jq -n --arg for proper JSON escaping
  • Added # prefix to PR number for better readability

Fixes #4879.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] .github/workflows/pr-merge.yaml does not include title or link Fix pr-merge.yaml to include PR title and link in Discord notifications Aug 28, 2025
Copilot AI requested a review from netmindz August 28, 2025 06:53
@netmindz netmindz marked this pull request as ready for review August 28, 2025 07:42
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Aug 28, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbit review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbit in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbit in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbit gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbit read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Join our Discord community for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbit help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbit ignore or @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbit summary or @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbit or @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@netmindz netmindz merged commit f15c1fb into main Aug 28, 2025
40 checks passed
@netmindz netmindz deleted the copilot/fix-4879 branch December 9, 2025 18:58
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.

.github/workflows/pr-merge.yaml does not include title or link

2 participants