Skip to content

Conversation

@just-boris
Copy link
Member

Issue #, if available:

Description of changes:

This metric is redundant, because it is basically a duplicate of logComponentUsed call, containing the very same data.

But this metric causes us issue on the receiver's end, because it produces "fake components" in the adoption metrics, which requires workarounds.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov
Copy link

codecov bot commented Jan 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.22%. Comparing base (5ae5804) to head (8028676).
⚠️ Report is 47 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
- Coverage   99.23%   99.22%   -0.01%     
==========================================
  Files          27       27              
  Lines         652      649       -3     
  Branches      173      170       -3     
==========================================
- Hits          647      644       -3     
  Misses          5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@just-boris just-boris marked this pull request as ready for review January 21, 2025 17:59
@just-boris just-boris requested a review from a team as a code owner January 21, 2025 17:59
@just-boris just-boris requested review from pan-kot and removed request for a team January 21, 2025 17:59
@just-boris just-boris added this pull request to the merge queue Jan 22, 2025
@just-boris just-boris removed this pull request from the merge queue due to a manual request Jan 22, 2025
@just-boris
Copy link
Member Author

This metric is still used in our business processes. Too difficult to change, let's keep it as is

@just-boris just-boris closed this Oct 22, 2025
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