search

bluwy / publint

Lint packaging errors
https://publint.dev
MIT License
932 stars 19 forks source link

Consider dropping EXPORTS_GLOB_NO_DEPRECATED_SUBPATH_MAPPING when it occurs before an equivalent `"./*"` #62

Closed andrewbranch closed 11 months ago

andrewbranch commented 11 months ago

https://publint.dev/tslib@2.6.1 issues a warning on the export subpath "./", which we added and kept for whatever versions of Node (probably EOL and/or patched now) supported the trailing slash and not the trailing star. tslib also declares an equivalent "./*" key before the "./" key, so the latter one, while deprecated, is kept intentionally and isn’t doing any harm.

bluwy commented 11 months ago

I think we can definitely lower it as a "suggestion" type if it's only for compat and has the adjacent key. I'd like to keep the message still since the trailing slash support has been long EOL, and the suggestion could be kept as a nudge for the next major.