Automattic / Automattic/harper

False positive: "It's time" flagged to change to "Its time" by the `Its Possessive` linter

Open
#3,219 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)
106

Description

Image

Technically both :"its time" and "it's time" are grammatical, but as a whole sentence it's clear that "It's time" as a contracted form of "It is time" is what's intended.

The solution would be to not flag either. A special case might be needed to exclude this, or maybe just knowing that it's the beginning to the end of a sentence?

Contributor guide

Open the contributing guide

Research direction

No file or test is named. Reproduce the warning for sentence-initial "It's time", then locate the `Its Possessive` rule and its existing tests. Done means the intended contracted form is no longer falsely flagged while genuine possessive uses remain 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
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.