Automattic / Automattic/harper
False positive: "It's time" flagged to change to "Its time" by the `Its Possessive` linter
Open
bug
harper-core
linting
- Dominant language
- Rust
- Stars
- 15.4k
- Forks
- 627
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 106
Description
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
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