Move tslint to devDependencies list

[lukeed]

Hey there,

I'm not very familiar with this module (sorry!) but I discovered that it's pulling in tslint as a dependency, which makes it significantly harder/impossible to use in modern browsers with ESM.

And in Node.js projects, it forces tslint (and all that comes with it) to be installed ... even though it appears to do nothing.

Thanks!

[zikaari]

Please see if this fixes the bundling issue - shikijs/shiki#22 (comment)

Will push the patch, over the weekend sometime. cheers.

[lukeed]

I know it can be bypassed. It just shouldn't be there.

[octref]

Hey @NeekSandhu, do you mind merging #22 and make this happen?

I'm in the process of reducing shiki's size, and tslint dependency is 3MB.

[zikaari]

@octref @lukeed apologies for the slack guys. will get this sorted tonight 👍

[zikaari]

2.2.5 is now live, can you please double-check if everything's nominal?

[octref]

Yes everything's normal, thanks!