Problem/Motivation

The CSpell job in the current 8.x-1.x baseline fails as an allowed failure, so the pipeline reports passed with warnings.

Exact CI findings

Pipeline 911674, job 11344553 reports 46 findings in 11 files and 20 distinct unrecognized words: auther, checkboxeses, cmail, commment, customised, gnuget, greggles, haschecked, knaddison, languates, mailalert, petrakis, recieve, stefanos, stefanospetrakis, subscriped, taht, tnid, uinit, and umail.

Affected files: comment_notify.install, comment_notify.migrate.inc, comment_notify.module, comment_notify.permissions.yml, INSTALL.TXT, README.md, src/Form/CommentNotifySettings.php, src/Hook/CommentNotifyHooks.php, tests/src/Functional/CommentNotifyConfigPageTest.php, tests/src/Functional/CommentNotifyUserPreferencesTest.php, and tests/src/Kernel/CommentNotifyTokenReplaceTest.php.

Proposed resolution

Correct genuine misspellings and API method casing. Add a project CSpell configuration/dictionary only for valid project-specific identifiers, contributor names, and legacy data aliases that must remain unchanged.

Testing instructions

  1. Run CSpell with the Drupal GitLab template configuration.
  2. Run the complete Comment Notify test suite.
  3. Confirm the CSpell CI job passes, rather than failing as an allowed failure.

Relevant CI

AI use disclosure

AI-Generated: Yes. OpenAI Codex was used to inspect the current CI output, draft this issue, and will assist with implementation and verification. All changes and test results will be reviewed by a human before merge.

Issue fork comment_notify-3614937

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:

About issue forks

Comments

gnuget created an issue. See original summary.

gnuget opened merge request !25

gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Active » Needs review

Validation complete.

Merge request: https://git.drupalcode.org/project/comment_notify/-/merge_requests/25
Pipeline: https://git.drupalcode.org/project/comment_notify/-/pipelines/911698
Targeted CSpell job: https://git.drupalcode.org/project/comment_notify/-/jobs/11344770 — passed.

The complete Comment Notify test directory was also run locally in DDEV and completed successfully. No merge was performed.

AI-Generated: Yes. OpenAI Codex assisted with implementation, test execution, CI inspection, and drafting this validation update. The change remains subject to human review before merge.

  • gnuget committed 2908aebf on 8.x-1.x 
    task: #3614937 Resolve CSpell findings in Comment Notify CI

By: gnuget
gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Needs review » 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.

Status: Fixed » Closed (fixed)

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