payloadcms / payload

Payload is the open-source, fullstack Next.js framework, giving you instant backend superpowers. Get a full TypeScript backend and admin panel instantly. Use Payload as a headless CMS or for building powerful applications.
https://payloadcms.com
MIT License
24.94k stars 1.59k forks source link

Upgrade `file-type` #8111

Closed benmccann closed 1 month ago

benmccann commented 1 month ago

Link to reproduction

https://github.com/payloadcms/payload/blob/08fdbcacc02c1768770516c2dd2e22df7bc48843/packages/payload/package.json#L97

Environment Info

beta.100

Describe the Bug

payload 3 currently uses file-type 17 while the latest is file-type 19. The newer version is smaller and would remove a couple of transitive dependencies

Reproduction Steps

See https://github.com/payloadcms/payload/blob/08fdbcacc02c1768770516c2dd2e22df7bc48843/packages/payload/package.json#L97

Adapters and Plugins

No response

AlessioGr commented 1 month ago

I'll upgrade this to file-type 19.3.0. Cannot upgrade to the latest (file-type 19.4.0 and higher) because of the following:

Damn I wish there was a way to add comments to package.json's to document these decisions

benmccann commented 1 month ago

that's awesome!! thank you!

Damn I wish there was a way to add comments to package.json's to document these decisions

there is, but it's really hacky...

"comment": "we use version 19.3 of file-type because..."
AlessioGr commented 1 month ago

Fixed by https://github.com/payloadcms/payload/pull/8195

there is, but it's really hacky...

"comment": "we use version 19.3 of file-type because..."

Yeea that might be a bit too hacky - I'm hoping for jsonc support or something like that to be added in the future!

github-actions[bot] commented 1 month ago

🚀 This is included in version v3.0.0-beta.107

github-actions[bot] commented 1 month ago

This issue has been automatically locked. Please open a new issue if this issue persists with any additional detail.