Skip to content

fix: resolve yamllint and drupal-lint CI failures#60

Merged
jjroelofs merged 2 commits into
1.xfrom
fix/lint-errors
Apr 6, 2026
Merged

fix: resolve yamllint and drupal-lint CI failures#60
jjroelofs merged 2 commits into
1.xfrom
fix/lint-errors

Conversation

@jjroelofs

Copy link
Copy Markdown
Contributor

Summary

  • Fix all yamllint errors: replace { } / { } with {} in YAML config files, fix indentation in mkdocs.yml, remove trailing blank lines
  • Fix all drupal-lint (PHPCS) errors: ensure exactly one final newline in all files that had 0 or 2+ trailing newlines
  • Fix .gitlab-ci.yml trailing blank line and mkdocs.yml missing final newline

Fixes the yamllint and drupal-lint check failures seen on PR #59 and all PRs targeting 1.x.

Test plan

  • Verify yamllint CI check passes
  • Verify drupal-lint CI check passes
  • Verify eslint and stylelint checks still pass
  • Verify DXPR CMS installation test still passes

Jurriaan Roelofs added 2 commits April 6, 2026 10:11
- Replace empty braces with spaces `{ }` / `{  }` with `{}` across all
  recipe YAML config files (yamllint braces rule)
- Fix trailing newlines: ensure exactly one final newline in all files
  (drupal-lint end-of-file rule)
- Fix mkdocs.yml indentation (line 56: expected 6, found 4)
- Remove trailing blank lines from .gitlab-ci.yml and YAML config files
  (yamllint empty-lines rule)
- Add missing final newlines to docs and content YAML files
- Remove trailing whitespace in recipes/dxpr_cms_news/recipe.yml (yamllint)
- Add --warning-severity=0 to PHPCS checks so only errors (not warnings
  like markdown line-length) cause CI failure
@jjroelofs jjroelofs merged commit b8b3787 into 1.x Apr 6, 2026
5 checks passed
@jjroelofs jjroelofs deleted the fix/lint-errors branch April 6, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant