Skip to content
Snippets Groups Projects

#3527481 Run validation jobs and fix spelling errors

Closes #3527481

Merge request reports

Merged results pipeline #511771 passed

Merged results pipeline passed for e6a48c1d

Test summary results are being parsed

Closed by quietonequietone 1 month ago (Jun 2, 2025 10:27pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
17 17 There are undoubtedly more sources that we should add to this list, but this makes a very good starting point.
18 18
19 19 ## Reserved Words
20
20 <!--- cspell:disable -->
  • I don’t love having this comment directly in the docs themselves. Can we ignore this entire file via our cspell job configuration so we don’t have to leave any trace in the .md file?

  • Can I ask why you don't like this ignore directive in the file? It is not shown in the rendered pages. Yes we can easily configure the job to ignore the entire file, but that risks having undetected spelling errors in the top half of the file. I'm not arguing, I just want to understand your view of why you think <!--- cspell:disable --> is not a good solution.

  • Please register or sign in to reply
  • Derek Wright left review comments

    left review comments

  • quietone added 1 commit

    added 1 commit

    • e7dd4db3 - fix spelling of typehint and mymodule

    Compare with previous version

  • quietone added 2 commits

    added 2 commits

    • 669c90f0 - Woohoo, defaultaction, mlsp, funstuff and findme
    • 69a9b481 - docblocks - use singular instead

    Compare with previous version

  • Jonathan Smith added 1 commit

    added 1 commit

    Compare with previous version

  • quietone added 2 commits

    added 2 commits

    Compare with previous version

  • Jonathan Smith changed title from #3527481 Run linting and validation jobs to #3527481 Run validation jobs and fix spelling errors
    changed title from #3527481 Run linting and validation jobs to #3527481 Run validation jobs and fix spelling errors
  • closed

  • Please register or sign in to reply
    Loading