Skip to content

Spellcheck PRs before merge. #237

@terryjreedy

Description

@terryjreedy

In spite of previous spelling fixes over the years, there have been, I recall, at least 3 spell fix PRs over the last several months. The problem is that we keep committing patches that add new doubled words and misspellings. For #6374, Guido, who does not like the code churn, said "Ideally spell checks would run on PRs.".

I presume people preparing the patches are using some combination of of python with re and some linux facility. I have no idea how many false positives such people have to ignore.

A separate issue is whether passing a spell check should be opt in or opt out.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions