Skip to content

Conversation

@voxpelli
Copy link
Contributor

This is an alternative to #225 where rather than using @types/simple-oauth2 in the published types we ensure at test time that the local types are indeed compatible with the published types of @types/simple-oauth2.

That way a consumer that wants to have access to the full types of simple-oauth2 can add @types/simple-oauth2 in their own project, knowing that it will be compatible

Checklist

This is an alternative to fastify#225 where rather than using `@types/simple-oauth2` in the published types we ensure at test time that the local types are indeed compatible with the published types of `@types/simple-oauth2`.

That way a consumer that wants to have access to the full types of `simple-oauth2` can add `@types/simple-oauth2` in their own project, knowing that it will be compatible
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcollina
Copy link
Member

Yes, this is much better!

@mcollina mcollina merged commit 75c17e5 into fastify:master Sep 4, 2023
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