Automattic / Automattic/harper
Edge case in `OrthographicConsistency`: Allow lowercase spelling of `test`
- 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
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