Skip to content

Fix grep warning in scripts/sync-version#2807

Merged
Kludex merged 2 commits intoKludex:mainfrom
ewie:fix-grep-warning
Feb 15, 2026
Merged

Fix grep warning in scripts/sync-version#2807
Kludex merged 2 commits intoKludex:mainfrom
ewie:fix-grep-warning

Conversation

@ewie
Copy link
Copy Markdown
Contributor

@ewie ewie commented Feb 3, 2026

Summary

Extended regular expressions (enabled via flag -E) don't support non-capturing groups under POSIX. Get rid of them as they don't add any value since we don't handle the captured groups in the first place.

The warning is:

$ ./scripts/sync-version
grep: warning: ? at start of expression
grep: warning: ? at start of expression
grep: warning: ? at start of expression
grep: warning: ? at start of expression
grep: warning: ? at start of expression
grep: warning: ? at start of expression

Checklist

  • I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.

Extended regular expressions (enabled via flag -E) don't support
non-capturing groups under POSIX.  Get rid of them as they don't add any
value since we don't handle the captured groups in the first place.
@Kludex Kludex merged commit 99f0d87 into Kludex:main Feb 15, 2026
19 checks passed
theseriff pushed a commit to theseriff/uvicorn that referenced this pull request Mar 9, 2026
Extended regular expressions (enabled via flag -E) don't support
non-capturing groups under POSIX.  Get rid of them as they don't add any
value since we don't handle the captured groups in the first place.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants