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

Remove two outdated test inputs from regex match timeout tests #106174

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

stephentoub
Copy link
Member

@stephentoub stephentoub commented Aug 9, 2024

Fixes #106096. These are too fast now and may trip over the (lack of) timeout sporadically.

@stephentoub
Copy link
Member Author

@ericstj, @jeffhandley, or @steveharter, could you please review this simple test change? Thanks.

Copy link
Member

@buyaa-n buyaa-n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ericstj ericstj merged commit d45f8b5 into main Aug 13, 2024
81 of 84 checks passed
@jkotas jkotas deleted the stephentoub-patch-1 branch August 16, 2024 04:46
@github-actions github-actions bot locked and limited conversation to collaborators Sep 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI failure RegularExpressions didn't throw expected TimeoutException
3 participants