Fix issues raised by cspell.

Issue fork reroute_email-3563274

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

liam morland created an issue. See original summary.

liam morland opened merge request !97

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Active » Fixed

All tests are passing.

Edit: Sorry, switched to wrong status.

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.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Fixed » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work

Agreed that it would be cleaner to change this via a text file or a cspell setting.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented

I think I tried that and "whitelist" is prohibited so the normal ways of ignoring it don't work.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Needs work » Needs review
paraderojether’s picture

paraderojether commented
Status: Needs review » Reviewed & tested by the community

Hi

I reviewed MR!97, and cspell is now passed on Gitlab CI.

Moving this to RTBC.
Thank you.

  • bohart committed e3824d87 on 2.3.x authored by liam morland 
    fix: #3563274 Fixed issues raised by the updated cspell configuration....
bohart’s picture

bohart
Location Lutsk, Ukraine
commented
Component: Code » Documentation
Priority: Normal » Minor
Status: Reviewed & tested by the community » Fixed

As it is related only to hook_update_N functions, `cspell:disable-next-line` might be an acceptable way (due to stopwords.txt file restrictions and ignorance of some words).

Merged into 2.3.x branch. Thanks.

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.