web icon indicating copy to clipboard operation
web copied to clipboard

fix: Export types from config-loader

Open Matsuuu opened this issue 2 years ago • 6 comments

What I did

  1. Added an export for types for the config-loader package

I was about to start using this package but had my typescript setup not finding the type declarations for it.

Turns out you have to add reference to the types as well, if you're declaring e.g. the import and require exports.

Source: https://webpack.js.org/guides/package-exports/#conditions

Should fix the "could not find a declaration file for module types" errors for this package

Not sure if there's other packages in this repo that could use this touch up too

Matsuuu avatar May 11 '23 13:05 Matsuuu

🦋 Changeset detected

Latest commit: 3419ca3ad2ee3bbb7c8949ebe58a5f603563f2db

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@web/config-loader Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar May 11 '23 13:05 changeset-bot[bot]

I think this should cause a patch level version bump to the package, I'll try n figure out how to set that up

Matsuuu avatar May 11 '23 13:05 Matsuuu

I'm like 100% that the tests aren't failing due to this change :D Could this get a look/merge?

Matsuuu avatar May 17 '23 09:05 Matsuuu

@Matsuuu can you bring this branch up to date with the latest changes on master? that should make CI pass

thepassle avatar Jun 08 '23 08:06 thepassle

@thepassle Sure can. Gimme a couple of hours and I'll push the latest

Matsuuu avatar Jun 08 '23 08:06 Matsuuu

Welp. Time escaped me last week. Did it now, let's see if the CI runs nicely

Matsuuu avatar Jun 12 '23 06:06 Matsuuu