gaurishhs / elysia-decorators

Decorators support plugin for Elysia
MIT License
67 stars 6 forks source link

Missing dependency with latest version #4

Closed awallat closed 10 months ago

awallat commented 10 months ago

Using the latest version of bun (1.0.21) and elysia, I cannot use the decorators anymore. The following error appears:

$ bun build --target bun src/index.ts --outdir ./dist
       return allDirs.concat(require('glob').sync(path.normalize(dir)));
                                           ^
error: Could not resolve: "glob". Maybe you need to "bun install"?

After adding "glob" to my package.json. It's working again.

gaurishhs commented 10 months ago

Yes, that's the intended behaviour. I probably didn't mention it in the readme but i did it here https://github.com/gaurishhs/elysia-decorators/blob/main/demo/index.ts#L2

gaurishhs commented 10 months ago

But i think i should migrate to Bun.Glob

awallat commented 10 months ago

Thanks for the information. Using Bun.Glob sounds like a good idea!