GitlabCI: Fix CSPELL validation errors

Quick follow-up on this issue:

A few additional commits were added to the current merge request MR!39 with a few comments, see above at #2.

But mostly, at this point:
Last build on MR!39: https://git.drupalcode.org/issue/colorbox-3467008/-/pipelines/248321

• The CSPELL job now seems to be passing ✅
  CSPELL job: https://git.drupalcode.org/issue/colorbox-3467008/-/jobs/2385512
   
• The PHPUnit Tests are passing as well 🟢 \o/
  PHPUnit job: https://git.drupalcode.org/issue/colorbox-3467008/-/jobs/2385513

We would greatly appreciate if a maintainer or someone with write permission could take a look at ticket's merge request and let us know if there would be any more work needed.

Feel free to let us know if you have any questions or concerns on merge request MR!39 or any aspect of this ticket in general, we would surely be glad to help.
Thanks in advance for your feedback and reviews.

Unfortunately the changes to the form elements and values makes this a more consequential and potentially breaking change. Eg anyone altering these forms will have to update code to match.

Probably better for now to ignore those spellings and open a follow up to rename these form elements with possible upgrade path if needed.

Otherwise, looks great.

Thanks!
-Derek

Thanks a lot Derek (@dww) for the very helpful review and speedy feedback, it's greatly appreciated! 🙂

Unfortunately the changes to the form elements and values makes this a more consequential and potentially breaking change. Eg anyone altering these forms will have to update code to match.

Good catch! I had not thought about this one 😅

Changes to src/Form/ColorboxSettingsForm.php have been reverted and the words slideshowauto and slideshowspeed ignored in the file.

Another breaking change:
https://git.drupalcode.org/project/colorbox/-/merge_requests/39/diffs?di...
Otherwise, that made me think of another breaking change to the colorbox.install file where the word please was removed in a t function call, thus potentially breaking existing string translations:
I had a similar comment from Tim (@TR) at #3157138-19: GitlabCI: Fix CSPELL validation errors → , so disabling the rule ForbiddenWord for this line seems like a better option for now.

As suggested a follow-up task was created, see 📌 CSPELL: Fix spelling for breaking variables and/or identifiers Active in case maintainers would still like to proceed with these changes.

Once again, we would greatly appreciate your reviews, comments and feedback.
Feel free to let us know if you would have any questions or concerns on any of the changes in the merge request or this ticket in general, we would be glad to help.
Thanks in advance!

@DYDave, thanks for cleaning this up, and @dww, thanks for reviewing. I have merged this MR.

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