Nico Huber posted comments on this change.

View Change

Patch set 1:

(1 comment)

• File util/git-hooks/pre-push:

  • Patch Set #1, Line 34: \(\([0-9]\+\.\)\+\([0-9]\+\)\)$

    Outer parentheses: because that's needed when using the matching expression

    Same paragraph mentions ^ is implicit and unclear what happens
    if you use it. So better leave it out.

    Shorter alternative:

        \(\([0-9]\+\.\)\{2,\}x\)$

To view, visit change 21834. To unsubscribe, visit settings.