Feature/683 lint sticky header.js

[markconroy]

Closes #683

---

Thanks to Big Blue Door for sponsoring my time to work on this.

[millnut]

@markconroy this might be false, due to prettier needing to be in a higher level for local linting and instead flagging a false result. I think this is the related bug https://www.drupal.org/project/drupal/issues/3314585

The drupal core config actually wants single quote https://git.drupalcode.org/project/drupal/-/blob/11.x/core/.prettierrc.json?ref_type=heads#L4

[markconroy]

Darn. Thanks @millnut

Let's try figure this out in MT tomorrow.

[millnut]

@markconroy yep lets discuss in MT. On the github workflows we copy the prettier file to a location where it reads correctly as part of the eslint check.

cd html/web/core
cp -f .prettierrc.json ../.prettierrc.json
node ./node_modules/eslint/bin/eslint.js --ext .yml --resolve-plugins-relative-to=./web/core ../../${{inputs.project_path}}

The drupal.org GitLab CI does similar but uses a symlink instead before running the eslint check

$ test -e .prettierrc.json || ln -s $CI_PROJECT_DIR/$_WEB_ROOT/core/.prettierrc.json .
$ $CI_PROJECT_DIR/$_WEB_ROOT/core/node_modules/.bin/eslint --no-error-on-unmatched-pattern --ignore-pattern="*.es6.js" --resolve-plugins-relative-to=$CI_PROJECT_DIR/$_WEB_ROOT/core --ext=.js,.yml --format=junit --output-file=$CI_PROJECT_DIR/junit.xml $_ESLINT_EXTRA . || EXIT_CODE_FILE=$?

[finnlewis]

@markconroy wants to review this again.