Automattic / Automattic/harper

False positive: sentence capitalization in tables

Open
#2,391 0 comments 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?**

```md
| a table header |
| -------------------------------------------- |
| protection from accidental commits to happen |
```

`SentenceCapitalization`on `protection`

**Why is this incorrect?**
1. It is not a sentence.
2. Capitalization often is omitted in table cells, so it might make sense to disable`SentenceCapitalization` in tables entirely, or at least make that an option.

Contributor guide

Open the contributing guide

Research direction

Reproduce the Markdown table example, then trace the SentenceCapitalization rule and the entry point that handles table cells. Done means this example is no longer incorrectly flagged, either through table-aware behavior or a clearly defined table-specific option.

Written by the indexing model from the issue text.

Assessment

Tech stack
markdown, 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.