-
Notifications
You must be signed in to change notification settings - Fork 60
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: add skipType option #75
Conversation
The rest of the failing tests now just match the existing behaviour, they were either wrong or looked things looked fine as they were.
Could you please remove I do think it's good that it gets updated to the new lockfile format but it also seems like a typical way to include a malicious package. I think it might be a good idea to do it in a separate PR. 😁 |
Except for my two comments I think it looks great 😁 👍 |
Co-authored-by: Julius Recep Colliander Celik <julius.cc@hotmail.com>
a5c5fee
to
0f980fc
Compare
Oh, I must have somehow missed that lockfile being in there. I rolled that one back. |
Great work! Thanks for the PR ❤️ |
🎉 This PR is included in version 7.5.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Somehow, the skipped type still appears as
undefined
. I'll look back into finishing this once I get the time again.closes #74