- Issue created by @m4olivei
- Merge request !242Issue #3440301: Eslint is failing with ESLint couldn't find the config "airbnb-base" to extend from → (Open) created by m4olivei
- 🇪🇸Spain fjgarlin
Has “navigation” already been merged into core? If that’s the case i don’t think this issue would still be happening there (in the core’s eslint job).
- 🇨🇦Canada m4olivei Grimsby, ON
@fjgarlin correct, it was moved into core in ✨ Add the new Navigation to core as an Experimental module Fixed as an experimental module. And indeed, eslint is all good in that context.
Moving this issue back to the contrib module, although note that the contrib space is more or less abandoned for all intents and purposes. The core merge is still fresh, so the contib module is still not completely cleaned up and clear to the that effect. All that is to say, the navigation module contrib space might not be the best place to work on the eslint issue anymore?
- 🇪🇸Spain fjgarlin
Perfect. That's not a problem. This issue is still a good reference for investigation and I think this is the right queue. We have play with the MR with the current "contrib" version of the module, which is fine for our investigation.
- 🇪🇸Spain ckrina Barcelona
Just to point out that the contrib module is not going to be used anymore. So unless you want to do anything specific here for drupal.org or general contrib purposes, it'd be great to leave all issues closed to not give a false message to people :)
- Status changed to Closed: outdated
8 months ago 11:23am 30 April 2024 - 🇪🇸Spain fjgarlin
Agree. We can close and still use the MRs for the testing of #3438843: Problem with eslint extends → .
This issue doesn't need to be open at all for the above. - First commit to issue fork.