Problem/Motivation

There are a number of cases where Pwned is misspelt as Pnwed. We should replace them. We may need to support BC for class names.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork password_policy_pwned-3434833

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

kim.pepper created an issue. See original summary.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Active » Needs review

kim.pepper opened merge request !3

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Title: Fix 'pnwed' typos » Fix 'pnwed' typos and other linting errors
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Needs review » Fixed

Committed to 2.x

jonathan1055’s picture

jonathan1055 commented

I saw your comment on Slack, and having worked on the cspell job, I was interested in the problem. What did you do to fix the 'bin' directory?

Also I notice you have _CSPELL_WORDS: 'pwned,hibp' but one of the recent improvements, now rolled out to all users, is that any part of a module name is automatically added to the allowed list. So you should not need pwned in that list.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented

@jonathan1055:

What did you do to fix the 'bin' directory?

I removed that config so it went back to vendor/bin.

any part of a module name is automatically added to the allowed list. So you should not need pwned in that list.

oh, good to know!

Status: Fixed » Closed (fixed)

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