Skip to content

Adjust check-spelling pattern #2804

@lemeurherveCB

Description

@lemeurherveCB

On version 0.0.22 (and 0.0.21), we noticed that -latest is wrongly recognised as atest, see updatecli/updatecli#2758

As a workaround we're adding atest in our exclude list for now.

Maybe - is the culprit here?

It's coming from:

(?:^|[\t ,>"'`=(])-[DPWXYLlf](?=[A-Z]{2,}|[A-Z][a-z]|[a-z]{2,})

You'll want to adjust that pattern (the simplest thing is to remove the l)

-(?:^|[\t ,>"'`=(])-[DPWXYLlf](?=[A-Z]{2,}|[A-Z][a-z]|[a-z]{2,}) 
+(?:^|[\t ,>"'`=(])-[DPWXYLf](?=[A-Z]{2,}|[A-Z][a-z]|[a-z]{2,}) 

It's possible for check-spelling to identify which patterns produced a fractional term, but it's somewhat expensive (it'd involve running all the patterns against the line until it identifies just the ones that contribute).

I'm not sure if @prerelease would improve this case (I'm currently cleaning up all of the commits I have in prerelease... hoping to have a clean test to release really soon).

Originally posted by @jsoref in #2758 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workinggithub_actionsPull requests that update Github_actions codegood first issueGood for newcomers

    Type

    No type
    No fields configured for issues without a type.

    Projects

    Status
    Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions