-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update npm deps #64
Update npm deps #64
Conversation
Signed-off-by: Sora Morimoto <sora@morimoto.io>
Signed-off-by: Sora Morimoto <sora@morimoto.io>
You can confirm this with Knip:
{
"$schema": "https://unpkg.com/knip/schema.json",
"workspaces": {
".": {
"entry": ["src/index.ts", "src/*.test.ts"],
"project": "src/*.ts"
},
"example": {
"entry": ["index.ts", "*.test.ts"],
"project": "*.ts"
}
}
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a question, but looks good to me!
This change is not included in the latest point release, but are you planning to release this? |
@smorimoto it will go out with the next release release. |
@febuiles Could you publish a new point release for this? |
@smorimoto If I push the PR I'll need to get someone's approval, can you update the version and push a PR? I can then be the approver and do the release. |
@febuiles Sure! |
Correct unused or unlisted dependencies.
Although it was not intended, this reduces the size of the package 🤷♂️
Before
After