Automattic / Automattic/harper
False Positive: `incl. public` -> `incl. Public` (`SentenceCapitalization`)
Open
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
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