Skip to content

Remove relay debug tracing - #64

Closed
HeyItsGilbert wants to merge 1 commit into
mainfrom
chore/remove-relay-trace
Closed

Remove relay debug tracing#64
HeyItsGilbert wants to merge 1 commit into
mainfrom
chore/remove-relay-trace

Conversation

@HeyItsGilbert

Copy link
Copy Markdown
Member

Removes temporary [DEBUG-pr-relay] logs after confirming an eligible GitHub ready_for_review delivery reaches Discord successfully with HTTP 204.

Verification: bun test tests/github-pr-discord.test.mjs and pwsh ./build.ps1 -Quiet -Task Test.

Copilot AI lite review requested due to automatic review settings August 30, 2026 02:13
@netlify

netlify Bot commented Aug 30, 2026

Copy link
Copy Markdown

Deploy Preview for psake ready!

Name Link
🔨 Latest commit a1c8d16
🔍 Latest deploy log https://app.netlify.com/projects/psake/deploys/6a9391dc8c3abc000806d246
😎 Deploy Preview https://deploy-preview-64--psake.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes temporary relay debug tracing from the Netlify function that forwards eligible GitHub pull_request webhook deliveries to Discord, now that end-to-end delivery has been validated.

Changes:

  • Removed conditional [DEBUG-pr-relay] console.info logs gated by DEBUG_PR_RELAY before and after the Discord webhook POST.
  • Leaves the existing error logging and non-error response behavior intact.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

2 participants