feat: no harm no fowl→no harm no foul

[hippietrail]

Issues

N/A

Description

While working on the "neither fish nor fowl" linter I noticed people also mix up "no harm no foul".

In this case one or both "no" might wrongly have "nor" as well as maybe having "fowl" instead of "fowl".

This linter corrects all wrong forms to "no harm no foul" or "no harm, no foul", depending on whether the original used a comma or not.

Technically the comma is required, but that's not enforced. We could add it as an option if and when linters can have options.

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.

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.