Skip to content

Add DO_NOT_TRACK env and mock http/https in tests#8

Merged
willwashburn merged 1 commit into
mainfrom
dontsendeventsinci
Feb 9, 2026
Merged

Add DO_NOT_TRACK env and mock http/https in tests#8
willwashburn merged 1 commit into
mainfrom
dontsendeventsinci

Conversation

@willwashburn
Copy link
Copy Markdown
Member

@willwashburn willwashburn commented Feb 9, 2026

Set DO_NOT_TRACK='1' for the Test steps in CI, deploy, and publish workflows to disable telemetry during CI runs. Update telemetry unit test to add a posthog request mock and mock node:http/node:https so tests don't perform real network requests and can simulate request end events for deterministic behavior.


Open with Devin

Set DO_NOT_TRACK='1' for the Test steps in CI, deploy, and publish workflows to disable telemetry during CI runs. Update telemetry unit test to add a posthog request mock and mock node:http/node:https so tests don't perform real network requests and can simulate request end events for deterministic behavior.
@willwashburn willwashburn requested review from Copilot and khaliqgant and removed request for khaliqgant February 9, 2026 19:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@willwashburn willwashburn requested a review from Copilot February 9, 2026 19:16
@willwashburn willwashburn merged commit 769ad56 into main Feb 9, 2026
5 of 7 checks passed
@willwashburn willwashburn deleted the dontsendeventsinci branch February 9, 2026 19:30
@willwashburn willwashburn review requested due to automatic review settings March 23, 2026 19:16
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