Skip to content

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Nov 7, 2025

temporarily disables Mixpanel telemetry. This project is taking too much of the event quota. In a followup PR, we will change to Clickhouse, since we are just doing a SUM of events and not doing any complex analysis/funnels/flows/etc.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Nov 7, 2025
@codecov
Copy link

codecov bot commented Nov 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #336      +/-   ##
==========================================
- Coverage   52.36%   52.18%   -0.18%     
==========================================
  Files          32       32              
  Lines        3516     3516              
==========================================
- Hits         1841     1835       -6     
- Misses       1675     1681       +6     
Files with missing lines Coverage Δ
comfy_cli/tracking.py 67.56% <100.00%> (-6.76%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@robinjhuang robinjhuang merged commit c5986e6 into main Nov 7, 2025
13 of 14 checks passed
@robinjhuang robinjhuang deleted the disable-telemetry branch November 7, 2025 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants