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
Remove @types/lodash.sortBy from project #1937
Conversation
Thanks for the contribution, @wasimTQ, and welcome to Openverse! To complete this PR, you'll need to update |
Got it. I was wondering why it's failing. I forgot to add the lock file. My bad!! |
82ddeab
to
2654091
Compare
@obulat @sarayourfriend When I ran the precommit locally, it worked. I don't know why it's failing on PR. Should I remove the type and use it as default export as it mentioned? |
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.
The lock file should not contain so many changes. The pnpm version mismatch might have caused this, but I'm not entirely sure.
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.
The lock file appears to have changes for a lot of unrelated packages, which causes failures in the linting.
When I removed @types/lodash.sortby
locally, and ran just node-install, I only saw 4 changes in
pnpm-lock.yamlfile. Could you check the version of
pnpmyou are using? It should match the version in
package.json`'s engines property to generate the correct lock file.
Here's the diff I see in the lock file:
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index e80070fab..8b2b351ea 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -55,7 +55,6 @@ importers:
'@testing-library/vue': ^5.8.2
'@types/express-useragent': ^1.0.2
'@types/jest': ^26.0.22
- '@types/lodash.sortby': ^4.7.7
'@types/module-alias': ^2.0.1
'@types/node': ^17.0.32
'@types/throttle-debounce': ^5.0.0
@@ -197,7 +196,6 @@ importers:
'@testing-library/vue': 5.8.2_rhqkolmkwunxzlyyxxsuwaiuri
'@types/express-useragent': 1.0.2
'@types/jest': 26.0.24
- '@types/lodash.sortby': 4.7.7
'@types/module-alias': 2.0.1
'@types/node': 17.0.32
'@types/throttle-debounce': 5.0.0
@@ -6390,12 +6388,6 @@ packages:
resolution: {integrity: sha512-1YXyYH83h6We1djyoUEqTlVyQtCfJAFXELSKW2ZRtjHD4hQ82CC4lvrv5D0l0FLcKBaiPbXyi3MpMsI9ZRgKsw==}
dev: true
- /@types/lodash.sortby/4.7.7:
- resolution: {integrity: sha512-J/4IS+jQopGBrrRetBXDCX0KnSeXJZ0rOTmGAxR9MWGV24YdHxX8IRi9LCGAU9GKWlBov9KRSfQpuup9PReqrw==}
- dependencies:
- '@types/lodash': 4.14.182
- dev: true
-
/@types/lodash/4.14.182:
resolution: {integrity: sha512-/THyiqyQAP9AfARo4pF+aCGcyiQ94tX/Is2I7HofNRqoYLgN1PBoOWu2/zTA5zMxzP5EFutMtWtGAFRKUe961Q==}
dev: true
@@ -11413,7 +11405,7 @@ packages:
resolution: {integrity: sha512-oWb1Z6mkHIskLzEJ/XWX0srkpkTQ7vaopMQkyaEIoq0fmtFVxOthb8cCxeT+p3ynTdkk/RZwbgG4brR5BeWECw==}
engines: {node: '>= 4.0'}
os: [darwin]
- deprecated: fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
+ deprecated: The v1 package contains DANGEROUS / INSECURE binaries. Upgrade to safe fsevents v2
requiresBuild: true
dependencies:
bindings: 1.5.0
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.
But it passes on local. Let me try after correcting the version
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.
Can you confirm it once the pipeline finish?
2654091
to
22381af
Compare
22381af
to
b3088ff
Compare
All the checks are 🟢 now! Thank you for your contribution, @wasimTQ 🎉 |
Fixes
Fixes #1930 by @sarayourfriend
Description
This PR removes
@types/lodash.sortBy
.Testing Instructions
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin