Conversation
Currently, when using the 0.0.55 with typescript 7, I get the following error: ``` Could not find a declaration file for module '@openbeta/sandbag'. 'C:/Users/remim/prog/climbhub/node_modules/.bun/@OpenBeta+sandbag@0.0.55/node_modules/@openbeta/sandbag/dist/sandbag.esm.js' implicitly has an 'any' type. There are types at 'C:/Users/remim/prog/climbhub/node_modules/.bun/@OpenBeta+sandbag@0.0.55/node_modules/@openbeta/sandbag/dist/index.d.ts', but this result could not be resolved when respecting package.json "exports". The '@openbeta/sandbag' library may need to update its package.json or typings.ts(7016) ``` It seems that although referenced by the "types" field, they are not exported, so typescript 7 refuses to use them. Adding "types" to export fixes it.
Coverage reportCaution Coverage does not meet threshold
Test suite run success268 tests passing in 17 suites. Report generated by 🧪jest coverage report action from 8eedf4c |
|
@ecstrema thanks for your contribution, the CI checks are failing because you have introduced a JSON syntax error |
|
This should now be fixed now. Sorry it was a single line so i didn't bother cloning. I edited directly in github's interface, which is far from good. |
|
Shot for the quick fix! I think you're using bun? I haven't checked but maybe they parse the package.json more leniently than yarn/node. The lint issue is a bit strange, I'll investigate that |
|
It's probably artifacts from the repo itself, but might be worth a better look. |
|
It might be simpler to add it to #191 though. |
|
Also, when are you planning the next release? I've been building my own library, but i'd prefer switching to this one to avoid maintaining it. |
Currently, when using the 0.0.55 with typescript 7, I get the following error:
It seems that although referenced by the "types" field, they are not exported, so typescript 7 refuses to use them.
Adding "types" to export fixes it.