Automattic / Automattic/harper

Flag mixing up "proof" and "prove"

Open
#2,583 0 comments 0 reactions 0 assignees View on GitHub
enhancement harper-core linting
Dominant language
Rust
Stars
15.4k
Forks
627
Avg merge
1d 15h
Merged PRs (30d)
106

Description

**The Error**

It's usually made by nonnative speakers who don't realize we have two words or just can't remember which one is which.

**Examples**
1. Giant Virus **Proofs** Life Evolved Differently Than We Thought

"proof" is typically a noun but it's also a verb that's not really rare, meaning "to proofread". This is why it's not included in the `NounVerbConfusion` linter.

Contributor guide

Open the contributing guide

Research direction

Start by locating the NounVerbConfusion linter and its existing tests, then review how it distinguishes nouns from verbs. Add coverage for the proof/prove confusion while preserving proof as a valid verb meaning to proofread; done means the example is flagged without introducing false positives for that usage.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.