Skip to content

feat: do diligence→due diligence/do due diligence#2810

Merged
elijah-potter merged 1 commit intoAutomattic:masterfrom
hippietrail:do-diligence
Feb 27, 2026
Merged

feat: do diligence→due diligence/do due diligence#2810
elijah-potter merged 1 commit intoAutomattic:masterfrom
hippietrail:do-diligence

Conversation

@hippietrail
Copy link
Collaborator

Issues

N/A

Description

I noticed that though we have some linting for mixing up "do" and "due", we do not flag "do diligence".

While looking into it I discovered that people write this not only instead of "due diligence" but also instead of "do due diligence".

I couldn't find a wide enough variety of usages to confidently build a proper linter with context to differentiate between the two. So instead made a Weir rule with a message that should hopefully allow the user to choose the right suggestion.

I hereby invite anyone to improve this linter with logic that can determine which of the two possible suggestions would be the right one.

How Has This Been Tested?

Two unit tests are included, one for each variant.

Checklist

  • I have performed a self-review of my own code
  • I have added tests to cover my changes

@elijah-potter elijah-potter added this pull request to the merge queue Feb 27, 2026
Merged via the queue into Automattic:master with commit def0606 Feb 27, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants