nextauthjs / next-auth

Authentication for the Web.
https://authjs.dev
ISC License
22.48k stars 2.98k forks source link

fix(types): re-export `TokenSet` type from `@auth/core/types` in framework pkgs #10755

Closed ndom91 closed 2 weeks ago

ndom91 commented 2 weeks ago

โ˜•๏ธ Reasoning

๐Ÿงข Checklist

๐ŸŽซ Affected issues

๐Ÿ“Œ Resources

vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Comments Updated (UTC)
auth-docs โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Apr 28, 2024 4:18pm
2 Ignored Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **next-auth-docs** | โฌœ๏ธ Ignored ([Inspect](https://vercel.com/authjs/next-auth-docs/5hxgeaTui9k7M7KpaixcorHCuo91)) | | | Apr 28, 2024 4:18pm | | **proxy** | โฌœ๏ธ Ignored ([Inspect](https://vercel.com/authjs/proxy/CZm97f8nmrSfdizT9zq7HCeot7CQ)) | | | Apr 28, 2024 4:18pm |
codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 5.26316% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 41.46%. Comparing base (7a6c2d3) to head (b8c39a0).

Files Patch % Lines
packages/next-auth/src/index.tsx 0.00% 16 Missing :warning:
packages/frameworks-solid-start/src/index.ts 0.00% 1 Missing :warning:
packages/frameworks-sveltekit/src/lib/index.ts 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10755 +/- ## ========================================== - Coverage 41.46% 41.46% -0.01% ========================================== Files 172 172 Lines 28236 28240 +4 Branches 1171 1177 +6 ========================================== + Hits 11709 11710 +1 - Misses 16527 16530 +3 ```

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

github-actions[bot] commented 2 weeks ago

Broken Link Checker

0 broken links found. Links organised below by source page, or page where they were found.

ndom91 commented 2 weeks ago

Closing this for now. TokenSet is more of an internal type and we'd love to rework the refresh token guide instead :pray: