Change babel/eslint-parser's inclusion of eslint-scope to be compatible with ESlint

[Standard8]

Q	A
Fixed Issues?
Patch: Bug Fix?	Y
Major: Breaking Change?	N
Minor: New Feature?	N
Tests Added + Pass?	Yes
Documentation PR Link
Any Dependency Changes?	Yes
License	MIT

eslint-scope is included as a dependency within @babel/eslint-parser using exact matching, i.e. only 5.1.0.

However since before ESLint version 7.5.0, ESLint has specified eslint-scope as ^5.1.0, and in more recent versions it is ^5.1.1. This means multiple versions can end up in the node dependency tree for those using ESLint with Babel, potentially causing conflicts or other issues.

This PR updates the dependency to match that of ESLint, so that the dependency tree for projects will only have one version of eslint-scope.

[babel-bot]

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/44778/

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 5d6f7a5:

Sandbox	Source
babel-repl-custom-plugin	Configuration
babel-plugin-multi-config	Configuration

[nicolo-ribaudo]

@arcanis Would something like require(require.resolve("eslint-scope", { paths: [require.resolve("eslint")] }) work? So that we can directly require ESLint's dependency without relying on deduplication.

[arcanis]

I don't remember if paths should be file or folder, but that should be fine either way. Another option is:

const {createRequire} = require(`module`);

const eslintReq = createRequire(require.resolve(`eslint`));
const eslintScope = eslintReq(`eslint-scope`);

[JLHwung]

In babel/babel-eslint#794 (comment) the maintainer of ESLint @mysticatea recommend @babel/eslint to depend on own eslint-scope. I think we should just use ^5.1.0 and do a major bump when eslint or eslint-scope is bumped.

[nicolo-ribaudo]

Thanks @JLHwung for linking that comment 👍

(@arcanis sorry if I pinged you out of the blue, I was worrying about PnP and completely forgot to mention it 😅)

[merceyz]

I was worrying about PnP

PnP is perfectly fine with that snippet, it's basically what babel does already to load plugins and presets

[nicolo-ribaudo]

Right, I didn't think about that!

[Standard8]

Thank you for the approvals & merge.