Replace globby to glob

[arowana-official]

Due to using old globby version I am getting some type error like this

Error: Cannot find symbol for node "IOptions" in "minimatch.IOptions" from packages/template-hardhat/node_modules/@types/glob/index.d.ts

It should be replaced to glob since it supports both CJS and ESM ( which is suitable for current hardhat version )

[cgewecke]

@arowana-official

• Which hardhat version are you using?
• Is your project ESM?

UPDATE

It looks like globby supports ESM based on this comment in their repo: sindresorhus/globby#197 (comment). It's possible bumping the dep to latest would be enough.

[arowana-official]

@cgewecke Yes but hardhat runs CJS internally so if we bump it it would simply break hardhat ( for V2, V3 is ESM by native though )

So I think it would make sense to replace to glob if not heavily rely on globby features.

[cgewecke]

@arowana-official Apologies, I don't understand how glob supports both CJS and ESM but globby doesn't. Could you explain this?

Could you also print the complete stack trace of the error you're seeing?

[arowana-official]

@cgewecke Sure,

For globby that supports ESM only sindresorhus/globby#193 ( and same with other sinresorhus projects )

Glob supports both as you could check the exports, main field for package.json https://github.com/isaacs/node-glob/blob/main/package.json

[cgewecke]

@arowana-official Thanks for that link..will take a look at swapping these.