Problem/Motivation

Three CI jobs pass with warnings on every MR (see pipeline #877836 on MR !20):

  • cspell: 201 issues in 45 files: technical terms with no project dictionary (LiteLLM, Ollama, chartjs...), a few British spellings, and two leftover French strings in test code.
  • eslint: 52 errors in the three js/ files (no-var, prettier formatting, undeclared Chart global...).
  • stylelint: the module ships a copy of core's .stylelintrc.json whose extends cannot resolve from the module directory, so the job dies on ConfigurationError before linting anything.

Proposed resolution

  • Add .cspell-project-words.txt (picked up automatically by the CI) and fix the misspellings in code comments.
  • Fix the eslint errors in js/.
  • Delete .stylelintrc.json so the job falls back to core's config, then fix whatever CSS violations surface.

Remaining tasks

  • None. Merged in 803b057, pipeline fully green on 1.0.x.

AI assistance

Root causes identified from CI job traces and fixes implemented with AI assistance.

Comments

codeitwisely created an issue. See original summary.

  • c980b283 committed on 1.0.x-3608770-ci-all-green
    Issue #3608770 by codeitwisely: Fix cspell, eslint and stylelint CI...

  • codeitwisely committed 803b0575 on 1.0.x
    Issue #3608770: Fix cspell, eslint and stylelint CI warnings
    

  • codeitwisely committed 803b0575 on 1.0.x-3604247-replace-reflectionproperty-reroute
    Issue #3608770: Fix cspell, eslint and stylelint CI warnings
    
codeitwisely’s picture

Issue summary: View changes
codeitwisely’s picture

Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

codeitwisely’s picture

Component: Miscellaneous » Code

Status: Fixed » Closed (fixed)

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