Skip to content

feat: fix redundant "first invented" / "invented for the first time"#3790

Open
hippietrail wants to merge 3 commits into
Automattic:masterfrom
hippietrail:redundant-firsts-1129
Open

feat: fix redundant "first invented" / "invented for the first time"#3790
hippietrail wants to merge 3 commits into
Automattic:masterfrom
hippietrail:redundant-firsts-1129

Conversation

@hippietrail

Copy link
Copy Markdown
Collaborator

Issues

Fixes #1129

Description

Flags verbs that imply the first time redundantly used with the word "first":

  • Invented for the first time
  • first invented

Some verbs make this implication with much more certainty, while others depend more on the context, so the message varies based on the verb.

The current patterns only cover the examples from the GitHub issue rather than trying to cover a wide range of hypothetical patterns. Running this on the fuzzer could be quite helpful.

So the chance of false negatives is reasonable. File bugs or comment here for false positives and negatives.

How Has This Been Tested?

cargo test

AI Disclosure

  • I am a human and didn't use any AI.
  • I used LLM features of my editor, but not an agent.
  • I used an AI agent interactively.
  • I am an agent or I got an agent to do the work autonomously.

I asked some AI for some coding idea but wrote all the code here myself.

If Your PR Implements or Enhances a Linter

  • I made up the sentences in the unit tests.
  • The sentences in the unit tests were generated by an AI.
  • I'm using examples from the bug report / feature request.
  • I collected real-world sentences for the unit tests.

Checklist

  • I have performed a self-review of my own code
  • I have added tests to cover my changes
  • I have considered splitting this into smaller pull requests.

@hippietrail hippietrail added enhancement New feature or request rust Pull requests that update Rust code linting labels Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request linting rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flag redundant firsts

1 participant