fix: added module path for types to the exports map

[AdrianGonz97]

The exports map in package.json was missing it's types module path, causing the import of @testing-library/svelte to throw a type error whenever users had "moduleResolution": "nodenext" set in their tsconfig.json.

[AdrianGonz97]

Additionally, I'm pretty sure this import path: https://github.com/testing-library/svelte-testing-library/blob/a7ad7fd969f2dc0d0cc70bb81539991a8c77f327/types/index.d.ts#L6-L6C71 should be 'svelte' rather than 'svelte/types/runtime'. I mention this because this is no longer a valid path in Svelte 4. Importing directly from the 'svelte' path is supported in both version.

I can add it to this PR if that's alright.

[yanick]

Yes, please add it!

…

On Sun, 25 Jun 2023, at 3:58 PM, CokaKoala wrote: Additionally, I'm pretty sure this import path: https://github.com/testing-library/svelte-testing-library/blob/a7ad7fd969f2dc0d0cc70bb81539991a8c77f327/types/index.d.ts#L6-L6C71 should be `'svelte'` rather than `'svelte/types/runtime'`. I mention this because this is no longer a valid path in Svelte 4. Importing directly from the `'svelte'` path is supported in both version. I can add it to this PR if that's alright. — Reply to this email directly, view it on GitHub <#228 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAE34WW2KRPZXO4EWVSDGLXNA7XLANCNFSM6AAAAAAZTFERMM>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[AdrianGonz97]

@yanick Done 👍

[yanick]

Merged, thanks!

[pngwn]

How frequently do you cut releases? My CI is currently failing because of a malformed exports map (no ./ at the start), would be good to get this out as soon as possible.

[yanick]

I just need to have a pr with the right keyword. Will try to do that later on today if i can

[github-actions]

🎉 This PR is included in version 4.0.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀