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

PatternAnalyzer and PatternTokenizer should accept an array of RegexFlags #10748

Closed
heaths opened this issue Mar 20, 2020 · 0 comments · Fixed by #11461
Closed

PatternAnalyzer and PatternTokenizer should accept an array of RegexFlags #10748

heaths opened this issue Mar 20, 2020 · 0 comments · Fixed by #11461
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Search
Milestone

Comments

@heaths
Copy link
Member

heaths commented Mar 20, 2020

Because it's expected that multiple RegexFlags may be used in PatternAnalyzer and PatternTokenizer, the flags property should be an array of RegexFlags that de/serialize as a |-delimited string.

@heaths heaths added Client This issue points to a problem in the data-plane of the library. Search labels Mar 20, 2020
@heaths heaths added this to the [2020] April milestone Mar 20, 2020
@heaths heaths self-assigned this Mar 20, 2020
@heaths heaths modified the milestones: [2020] April, [2020] May Apr 6, 2020
heaths added a commit to heaths/azure-sdk-for-net that referenced this issue Apr 20, 2020
heaths added a commit that referenced this issue Apr 21, 2020
* De/serialize |-delimited array of RegexFlags

Fixes #10748

* Declare related read-only collection properties

* Update public APIs

* Resolve PR feedback
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Search
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant