Automattic / Automattic/harper

Edge case in `OrthographicConsistency`: Allow lowercase spelling of `test`

Open
#2,447 1 comment 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)
106

Description

**What got flagged?**
`It's possible to test a specific function:`

**Why is this incorrect?**
Harper error for the word `test`:
> This word's canonical spelling is all-caps. (Harper OrthographicConsistency)

I think lowercase `test` should be allowed in this context.

**Example of correct usage:**
The flagged text is a good example.

---

Unrelated fun fact: I found this error when I wrote about what I learned while contributing a PR to fix #2446 :smile:

Not sure how to solve this, so feel free to fix it if you know a good approach.

Contributor guide

Open the contributing guide

Research direction

Start by locating the OrthographicConsistency implementation and any tests covering the quoted sentence, then reproduce the Harper error for lowercase “test” in that context. Done means the example is accepted while the existing all-caps canonical-spelling behavior remains intact, with a regression test for the edge case.

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.