search

blakeembrey / change-case

Convert strings between camelCase, PascalCase, Capital Case, snake_case and more
MIT License
2.29k stars 97 forks source link

Fixing import #335

Closed siddtheone closed 9 months ago

siddtheone commented 9 months ago

Hi @blakeembrey ,

Please review

blakeembrey commented 9 months ago

No, this isn’t correct and I won’t be merging it. You can read more about how ESM and imports work in the node documentation.

In future PRs I would recommend providing context in your PR description detailing what’s wrong and what you’re trying to fix and why, not just pinging maintainers. There’s been a lot of spam to other repos lately trying to land useless PRs so it’s tempting to just close ones that look like this, but I do want to help and I can’t if I don’t have any context on what you’re trying to do and why.

blakeembrey commented 9 months ago

Specifically this part for docs: https://nodejs.org/api/packages.html#subpath-exports