Fix eslint test [#3455995]

Problem/Motivation

Let's fix the eslint test.

Issue fork drulma-3455995

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

eslint changes, plain diff MR !10
Check out this branch for the first time

Check out existing branch, if you already have it locally

About issue forks

Comments

Comment #1

20 June 2024 at 14:19

ptmkenny created an issue. See original summary.

Comment #2

20 June 2024 at 14:20

ptmkenny opened merge request !10

Comment #3

ptmkenny commented 20 June 2024 at 14:29

I commented out the empty mapping values https://git.drupalcode.org/project/drulma/-/merge_requests/10/diffs?comm..., but perhaps they should be deleted instead? It seems this is just a generic docker template, so if they are not being set, there's no reason to have the placeholders there.

Comment #4

ptmkenny commented 20 June 2024 at 14:32

I decided to delete the empty mappings in docker-compose.yml because the eslint rules on comment spacing are confusing and the mapping options aren't being used anyway (which is why eslint is flagging them).