Automattic / Automattic/harper

Harper sometimes suggests "side affect" over "side effect"

Open
#2,958 1 comment 1 reaction 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

**What got flagged?**
> I forgot to test the side effect that users are deleted when clearing data.

**Why is this incorrect?**
In this case, "side effect" serves as a noun.

Contributor guide

Open the contributing guide

Research direction

Search the grammar rules and regression tests for the reported sentence and the "side affect" suggestion. Start with the rule that treats "affect" and "effect" as interchangeable, then add a focused test showing that "side effect" is accepted as a noun and verify the incorrect suggestion no longer appears.

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
Quiet
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.