Automattic / Automattic/harper

Wrongly flags `guy’s` to change to `guy's` and claims it's due to `Capitalization`

Open
#2,516 0 comments 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)
102

Description

**What got flagged?**

> And that’s when I started to really like the cut of this guy’s jib.

Image

Typographical apostrophe is _more correct_ than ASCII apostrophe. It's an orthography issue for sure, like spelling and capitalization and hyphenation are. But it's *not* a capitalization issue.

Contributor guide

Open the contributing guide

Research direction

Reproduce the capitalization warning for the sentence containing `guy’s`, then trace the rule that suggests replacing the typographical apostrophe and labels the issue as `Capitalization`. Done means the apostrophe is no longer incorrectly flagged or categorized as capitalization, with the behavior covered by an appropriate regression test.

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.