Skip to content

Rename NSRegularExpression options #1165

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

Merged
merged 1 commit into from
Aug 10, 2017

Conversation

spevans
Copy link
Contributor

@spevans spevans commented Aug 9, 2017

  • NSMatchingOptions -> NSRegularExpression.MatchingOptions

  • NSMatchingFlags -> NSRegularExpression.MatchingFlags

- NSMatchingOptions -> NSRegularExpression.MatchingOptions

- NSMatchingFlags -> NSRegularExpression.MatchingFlags
@spevans
Copy link
Contributor Author

spevans commented Aug 9, 2017

@ianpartridge See if this looks ok

@ianpartridge
Copy link
Contributor

@swift-ci please test

@ianpartridge
Copy link
Contributor

Timed out - is this a known issue @shahmishal?

@spevans
Copy link
Contributor Author

spevans commented Aug 9, 2017

is the timeout total time or time since something was last written to the terminal? I wonder if we can make the tests more chatty rather than just doing the 10..20..30 etc

@alblue
Copy link
Contributor

alblue commented Aug 9, 2017

@swift-ci please test

1 similar comment
@ianpartridge
Copy link
Contributor

@swift-ci please test

@ianpartridge ianpartridge merged commit 5fd6130 into swiftlang:master Aug 10, 2017
@spevans spevans deleted the pr_nsmatchingoptions branch September 19, 2017 15:09
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.

3 participants