Don't require installing @typescript-eslint/parser

[lishaduck]

For some reason, PNPM makes me install @typescript-eslint/parser as a peer dependency of Sheriff, rather than doing so automatically as a dependency. This should be investigated and fixed.

[AndreaPontrandolfo]

Depending on the version of PNPM, the auto-install-peers flag can be default true or default false.

[lishaduck]

Depending on the version of PNPM, the auto-install-peers flag can be default true or default false.

It defaults to true, there's not a conflict. It's probably a bug in pnpm, but just making note of it in case there's a bug in packaging I need to cover in #161.

[lishaduck]

I'm going to close this, we can see if it's still relevant with ESLint 9 + TSESLint 8

[lishaduck]

For future reference, I think this was fixed in PNPM 9.7 (pnpm/pnpm#8323)

[AndreaPontrandolfo]

Indeed PNPM 9.5 was kind of a broken release.
I'll update to 9.7 in the next release, for sure.

[lishaduck]

Indeed PNPM 9.5 was kind of a broken release. I'll update to 9.7 in the next release, for sure.

This was a bug installing it in my project, not installing the sheriff monorepo itself.
PNPM 9 is probably a worthwhile upgrade though.