Fix ESLint on CI

The gitlab issue to fix this is #3438843: Problem with eslint extends →

Just a querstion. You have

  "extends": [
    "../../../../core/.eslintrc.json",
    "airbnb-base",
    "airbnb-typescript/base",
    "prettier"
  ]

But the core config file core/.eslintrc.json already extends "airbnb-base" so do you really need to include it too?
That wouldn't fix the overall problem, but I'm just curious, that's all.

I just checked without it and got this:

Oops! Something went wrong! :(

ESLint: 8.45.0

ESLint couldn't find the config "airbnb-base" to extend from. Please check that the name of the config is correct.

The config "airbnb-base" was referenced from the config file in "/web/core/.eslintrc.json".

If you still have problems, please stop by https://eslint.org/chat/help to chat with the team.

Yes it's exactly the same error. This will not be fixed in #3438843: Problem with eslint extends → as it's a bigger problem. I have already created a new issue #3438843: Problem with eslint extends → to investiagte it. Do you know the mechanism by which the core eslint loads airbnb-base? I presume it is from a /vendor and I know how 'extends' works when you give a path to a file. But I don't know how a pre-built pre-defined config is "found". I'm sure it is simple, I've just not had cause to research it until now.

Do you know the mechanism by which the core eslint loads airbnb-base?

The core/.eslintrc.json file uses it via "extends". The library is brought via yarn, see the entry for eslint-config-airbnb-base in core/package.json here: https://git.drupalcode.org/project/drupal/-/blob/11.x/core/package.json#L66

We are doing yarn install in the .composer-base job, so it should be available and found.

In #5 fgm said

I just checked without it ...

Do you mean without the projects .eslintrc.json file? I tried that in an earlier commit in the MR and the job ran (but gave warnings of course). But the message you give above is "The config airbnb-base was referenced from the config file in /web/core/.eslintrc.json" I did not get this error when I removed .eslintrc.json file. So I just want to check what we are looking at here.

To clarify, I meant "without the airbnb-base" line in the project .eslintrc.json, not "without the project .eslintrc.json".

Thanks that's helpful. It would imply that the airbnb config is not being installed in your pipelines, regardless of whether you have it in your custom config file. That does not help you, but it does mean that hopefully ew do not have something fundamentally wrong with our eslint job. I can test that hypothesis on another project.

But we also have a discrepancy, as I thought I had tested it without the extends. This commit in the MR
https://git.drupalcode.org/project/g2/-/merge_requests/14/diffs?commit_i...
produced this job
https://git.drupalcode.org/issue/g2-3438413/-/jobs/1241595
It did not fail on the airbnb-base problem.

Right, that one failed on '@typescript-eslint/parser instead. Not necessarily better, but at least different.

I did a few changes, so here's patch3438413-13.patch : I don't know how to handle multiple contributors with the MR flow.

I think that as long as you don't work on the same file as Jonathan (.gitlab-ci.yml file), you should be fine committing/pushing changes to the branch of the MR. You'll just need to click on "Get push access" and then follow the instructions in "Show commands" to get the branch locally.

I am not currently working on this so there should not be any clashes. I can do investigation on other projects to help with #3432261: Eslint on contrib is making incorrect recommendations → If/when I want to make a further change to .gitlab-ci.yml I will check back here with you first.

Strangely enough this morning the pipeline on an unrelated MR passed the ESLint step. Were there changes in CI ? https://git.drupalcode.org/project/g2/-/jobs/1309927

Not forgotten, but still no fix upstream in #3438843: Problem with eslint extends → six months later.