Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat/types-in-dependencies #370

Merged
merged 1 commit into from
Mar 16, 2023
Merged

feat/types-in-dependencies #370

merged 1 commit into from
Mar 16, 2023

Conversation

MasterKale
Copy link
Owner

@MasterKale MasterKale commented Mar 11, 2023

This PR moves some types dependencies into package.json "dependencies". This will primarily help pull in @simplewebauthn/typescript-types into more projects without developers needing to manually npm install it as well as @simplewebauthn/server.

I installed this version with the repro included in #368 and believe that this will ensure types are always available:

Screenshot 2023-03-15 at 6 40 11 PM

Fixes #368.

@MasterKale MasterKale added package:browser @simplewebauthn/browser package:server @simplewebauthn/server labels Mar 11, 2023
@MasterKale MasterKale added this to the v7.2.0 milestone Mar 11, 2023
@MasterKale MasterKale merged commit 70f7e79 into master Mar 16, 2023
@MasterKale MasterKale deleted the feat/types-in-dependencies branch March 16, 2023 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:browser @simplewebauthn/browser package:server @simplewebauthn/server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add @simplewebauthn/typescript-types to dependencies of @simplewebauthn/server
1 participant