Automattic / Automattic/harper

False positive flagging to change "to rebased" to "to rebase"

Open
#1,993 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)
102

Description

**What got flagged?**

Image

> Bookmarks pointing to rebased commits will be

**Why is this incorrect?**
In this context "rebased" is acting as an adjective to qualify the noun "commits"

Contributor guide

Open the contributing guide

Research direction

Start by locating the grammar rule and tests responsible for changing "rebased" to "rebase". Reproduce the report using the sentence "Bookmarks pointing to rebased commits will be" and add a regression case showing that "rebased" is accepted as an adjective before "commits".

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.