Automattic / Automattic/harper

(milli) prefix should not be flagged if it precedes a unit of measurement?

Open
#2,745 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?**

"Use Date objects throughout instead of UNIX (milli)seconds"

Image

**Why is this incorrect?**

- "UNIX" should be in the default dictionary - see #1494
- the "milli" prefix in parentheses before "seconds" should not be flagged

**Example of correct usage:**

"Use Date objects throughout instead of UNIX seconds or milliseconds"

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the quoted sentence, including “UNIX (milli)seconds,” and locate the rule that flags the parenthesized prefix. Check the related #1494 discussion for the expected UNIX dictionary handling. Done means the prefix is not flagged before “seconds” and the example passes with the intended dictionary behavior.

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.