Skip to content

Fixup collector ID logic#80

Merged
grahamc merged 1 commit intomainfrom
fixup-traces
Mar 27, 2025
Merged

Fixup collector ID logic#80
grahamc merged 1 commit intomainfrom
fixup-traces

Conversation

@grahamc
Copy link
Member

@grahamc grahamc commented Mar 27, 2025

We were erroniously never setting the collector ID. The collector ID is intended to prevent one workflow run with multiple DS actions from reporting stack traces twice.

Description
Checklist
  • Tested changes against a test repository
  • Added or updated relevant documentation (leave unchecked if not applicable)
  • (If this PR is for a release) Updated README to point to the new tag (leave unchecked if not applicable)

We were erroniously never setting the collector ID.
The collector ID is intended to prevent one workflow run with multiple DS actions from reporting stack traces twice.
@netlify
Copy link

netlify bot commented Mar 27, 2025

Deploy Preview for detsys-ts-docs ready!

Name Link
🔨 Latest commit cf9fa3d
🔍 Latest deploy log https://app.netlify.com/sites/detsys-ts-docs/deploys/67e555ba1612fd00086d5232
😎 Deploy Preview https://deploy-preview-80--detsys-ts-docs.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 site configuration.

@grahamc grahamc merged commit 38df301 into main Mar 27, 2025
5 checks passed
@grahamc grahamc deleted the fixup-traces branch March 27, 2025 13:46
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