Skip to content

Conversation

@jhrozek
Copy link
Contributor

@jhrozek jhrozek commented Nov 26, 2025

The auth package had too many symbols exported. Let's export only what's needed and make most of the symbols internal to make the intent of how the package is to be used more explicit.

The auth package had too many symbols exported. Let's export only what's
needed and make most of the symbols internal to make the intent of how
the package is to be used more explicit.
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 77.41935% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.58%. Comparing base (b78ee5a) to head (665ae2b).

Files with missing lines Patch % Lines
internal/auth/mocks/mock_validator.go 0.00% 7 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #196   +/-   ##
=======================================
  Coverage   66.58%   66.58%           
=======================================
  Files          60       60           
  Lines        3765     3765           
=======================================
  Hits         2507     2507           
  Misses       1101     1101           
  Partials      157      157           

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

@rdimitrov rdimitrov merged commit 7ee2a6f into main Nov 26, 2025
11 checks passed
@rdimitrov rdimitrov deleted the auth-cleanup branch November 26, 2025 14:28
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.

5 participants