Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

semver: Update FAQ entry on regular expressions #998

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

har7an
Copy link

@har7an har7an commented Jan 20, 2024

with a third variant that works without non-capturing groups and is compatible with POSIX regex (i.e. for use in bash). This originates from an issue I opened earlier, found here: #981

There is a small repo which contains the terse form of the regex along with a more human-readable one and a bash script that demonstrates and runs the regex against the set of test strings from regex101. I also went and created a snippet for this regex on regex101.

I only just noticed that there is a long-standing open PR which adds exactly the same regex (but with less explanation) to the FAQ.

with a third variant that works without non-capturing groups and is
compatible with POSIX regex (i.e. for use in bash).
@jwdonahue
Copy link
Contributor

I only just noticed that there is a #724 which adds exactly the same regex (but with less explanation) to the FAQ.

Doh! I forgot all about that, sorry.

@har7an
Copy link
Author

har7an commented Feb 20, 2024

@jwdonahue Any news on this?

@jwdonahue
Copy link
Contributor

@har7an

I think we're waiting on the maintainers.

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.

None yet

2 participants