Automattic / Automattic/harper

The unit of time expander linter shouldn't always suggest plurals even when the number is not "1"

Open
#2,019 1 comment 0 reactions 0 assignees View on GitHub
bug linting
Dominant language
Rust
Stars
15.4k
Forks
627
Avg merge
1d 15h
Merged PRs (30d)
106

Description

**Describe the bug**

Image

> **Sub 100-ms response times** with SQLite FTS5 search

In a context like this we would write "Sub 100 millisecond response times" and not "Sub 100 milliseconds response times".

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the unit-of-time expander linter on the quoted “Sub 100-ms response times” example and locate the rule responsible for expanding the unit. Done means the example uses singular “millisecond” after 100 while ordinary cases that require a plural still behave correctly.

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
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.