Automattic / Automattic/harper

False "New Sentence" with imperial shorthands if not using a word like "tall" or "long"

Open
#3,260 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 15h
Merged PRs (30d)
106

Description

**What got flagged?**
`Bright light in a 25 ft. circle`
For "25 ft. / circle"
(Harper believes a new sentence starts at the /

**Why is this incorrect?**
Imperial measurements can be, and formally are followed by a period. This does not mark a new sentence, which Harper understands, but only with words like "tall" or "long" and not "circle" or "radius".

It should be noted that this CAN mark the end of a sentence. Two periods (e.g. "It was 25 ft.. A perfect square.") is incorrect and should be flagged.

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the reported sentence with “25 ft. circle” and compare it with examples using “tall” or “long,” then inspect the grammar-checking rule that identifies sentence boundaries after imperial abbreviations. Done means the reported phrase is not flagged, while a genuine sentence ending after the measurement remains detectable, including the incorrect double-period case.

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
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.