add "bun" export condition and set "types" exports first as ts 4.7 requirement

porsager / postgres

Postgres.js - The Fastest full featured PostgreSQL client for Node.js, Deno, Bun and CloudFlare

The Unlicense

7.26k stars 262 forks source link

Closed lukeed closed 10 months ago

lukeed commented 10 months ago

Closes #692

Bun auto-matches a few conditions, including "worker" and "node". It looks for itself ("bun") first.

See: https://bun.sh/docs/runtime/modules#importing-packages

Also made types the first condition, which is required according to the TS 4.7 announcement blog

porsager commented 10 months ago

Sorry, I got to push a commit just before your PR :P Can you rebase to just have the types fix, then I'll merge.

lukeed commented 10 months ago

Haha i saw, slow UI :S

Should already be updated

porsager commented 10 months ago

Seems like it.. My merge button hasn't even updated yet either - still thinks there are merge conflicts..

Angelelz commented 10 months ago

Thank you for working on this so quickly!

trylovetom commented 10 months ago

thanks!