E275: whitespace after annotated `match` variable (2.8.0 release blocker)

[asottile]

this code was previously ok, but is now triggering E275 on the match line:

class Found(NamedTuple):
    y: int 
    match: Match[str]

[asottile]

this is also getting triggered for:

def f():
    matcher = {"true": True, "false": False}.__getitem__
    #    ^

[cdce8p]

Thanks for bringing this up! Didn't had time to look at it in detail just yet, but I think the regex pattern was missing some end of line lookahead for it to work:

- ^\s*\b(?:match|case)(\s*)(?=.*\:)
+ ^\s*\b(?:match|case)(\s*)(?=.*\:(?=\s*$|\s*#))

[asottile]

that doesn't work either which is why I chose to revert rather than fix

match 0:
    case _: print('hi')

[cdce8p]

The new pattern would at least not create false-positives, although it couldn't detect an error in your example either.
Not sure there is an easy regex for all possible cases. match and case only being soft keywords complicates things a bit. Combine that with the ability to nest arbitrary patterns.

[cdce8p]

Opened #1033 with a second attempt at adding the whitespace checks. I've taken into account the previous comments and believe I've even found a solution to the inline case.