Automattic / Automattic/harper

False positive: `PronounVerbAgreement` is triggered by "you guys"

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

Description

Image

> Keep going please because I love you guys and want to keep watching

Apparently "guy" is a verb as well as a noun. Maybe the rule needs to be modified to check if the verb is also a noun, or maybe it needs a special case for "you" + "guys"

Contributor guide

Open the contributing guide

Research direction

No file or test is named. Start by locating the PronounVerbAgreement rule and reproducing the quoted “you guys” sentence; inspect its existing tests and behavior for ordinary pronoun–verb agreement. Done means this phrase is no longer flagged while genuine agreement errors remain detected.

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
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.