Skip to content

Remove useHistory - unused code#3207

Merged
yurishkuro merged 1 commit intojaegertracing:mainfrom
Parship12:rm-useHistory
Dec 11, 2025
Merged

Remove useHistory - unused code#3207
yurishkuro merged 1 commit intojaegertracing:mainfrom
Parship12:rm-useHistory

Conversation

@Parship12
Copy link
Copy Markdown
Member

@Parship12 Parship12 commented Dec 10, 2025

Which problem is this PR solving?

Checklist

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
@Parship12 Parship12 requested a review from a team as a code owner December 10, 2025 18:28
@Parship12 Parship12 requested review from jkowall and removed request for a team December 10, 2025 18:28
@Parship12 Parship12 marked this pull request as draft December 10, 2025 18:28
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.89%. Comparing base (1ceadb6) to head (0221357).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3207      +/-   ##
==========================================
- Coverage   97.89%   97.89%   -0.01%     
==========================================
  Files         262      261       -1     
  Lines        8212     8209       -3     
  Branches     2086     2142      +56     
==========================================
- Hits         8039     8036       -3     
  Misses        172      172              
  Partials        1        1              

☔ 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.

@Parship12 Parship12 marked this pull request as ready for review December 10, 2025 18:49
@yurishkuro yurishkuro added the changelog:refactoring Internal, non-functional code improvements label Dec 11, 2025
@yurishkuro yurishkuro merged commit c4fc9c2 into jaegertracing:main Dec 11, 2025
11 of 13 checks passed
@Parship12 Parship12 deleted the rm-useHistory branch December 11, 2025 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:refactoring Internal, non-functional code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants