Remove unnecessary exports

awharn commented 1 month ago

What It Does

Removes a set of exports that are not referenced outside of the related packages Deprecates some functions/classes that are identical and appear twice in the codebase

This PR is presently in a draft state to get team opinions on additional things that can be removed. All of the items removed here did not appear outside of their respective packages. However, GitHub search only works against V2 LTS.

How to Test

Review Checklist I certify that I have:

[x] tested my changes
[x] added/updated automated tests
[ ] updated the changelog
[x] followed the contribution guidelines

Additional Comments

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.18%. Comparing base (c12da19) to head (7ef1504). Report is 2 commits behind head on next.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## next #2159 +/- ## ========================================== - Coverage 91.22% 91.18% -0.05% ========================================== Files 629 628 -1 Lines 17912 17874 -38 Branches 3795 3692 -103 ========================================== - Hits 16341 16299 -42 - Misses 1570 1574 +4 Partials 1 1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.