Automattic / Automattic/harper

False Positive: `incl. public` -> `incl. Public` (`SentenceCapitalization`)

Open
#3,180 1 comment 1 reaction 0 assignees View on GitHub
bug harper-core linting punctuation
Dominant language
Rust
Stars
15.4k
Forks
627
Avg merge
1d 12h
Merged PRs (30d)
106

Description

**What got flagged?**
`incl. public` corrected to `incl. Public` by `SentenceCapitalization`.

**Why is this incorrect?**
The period (`.`) is being used to signify an abbreviation here, rather than the end of a sentence.

**Example of correct usage:**
"Use OOP (incl. public, private, and static variables)"

Contributor guide

Open the contributing guide

Research direction

Start by locating the SentenceCapitalization implementation and its existing tests, then reproduce the example where "incl. public" is changed to "incl. Public." Done means the abbreviation is no longer treated as a sentence boundary while other sentence-capitalization behavior remains covered by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
devtools
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.