I changed the YAML files basing on the ESLint errors reported by GitLab CI, thinking that the used .eslintrc file would not change the errors reported by ESLint. I then discovered that, removing that file, the reported errors were less.

If empty lines before comments in YAML files are not required by the default ESLint configuration used by GitLab, those empty lines need to be removed.

Issue fork examples-3459724

Command icon Show commands

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

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

Comments

apaderno created an issue. See original summary.

avpaderno’s picture

Status: Active » Needs review
avpaderno’s picture

Status: Needs review » Postponed

The GitLab CI pipeline has not reported any error about the empty lines before YAML comments I removed from some .yaml files as test.
I will remove all those empty lines, but I am postponing this issue to the other issues that would fix the left ESLint errors.

avpaderno’s picture

Title: Check if ESLint requires that an empty line is added before a comment in YAML files and remove those empty lines if they are not required » Remove empty lines before comments in YAML
Status: Postponed » Needs review

  • apaderno committed d85585c6 on 4.0.x
    Issue #3459724: Remove empty lines before YAML comments
    
avpaderno’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.