Skip to content
This repository was archived by the owner on Jun 24, 2021. It is now read-only.

Regex options#51

Merged
agcom merged 4 commits into
mainfrom
regex
Sep 19, 2020
Merged

Regex options#51
agcom merged 4 commits into
mainfrom
regex

Conversation

@agcom

@agcom agcom commented Sep 19, 2020

Copy link
Copy Markdown
Owner

End user changes

  • Fixed regex options, Regex options not fully supported #46.
  • Added builders for BsonRegularExpression using a Pattern
  • Changed decoders/encoders to use Pattern class instead of Regex
    • BsonInput.decodeRegularExpression now returns a Pattern instead of Regex
    • BsonOutput.encodeRegularExpression now accepts a Pattern instead of Regex

@agcom
WIP...Added PatternUtils to deal with regex flags/options
e4694de 
Added constructor like builder for `BsonRegularExpression` with `Pattern` as parameter

Added `BsonRegularExpression.toPattern` function
@agcom
WIP...Refactored defaultBsonModule declaration
6d7ee70 
Suppressed old `load` function `unused parameter type` warning
@agcom
WIP...Changed decodeRegularExpression signature to use Pattern
49fe6bc 
Changed `encodeRegularExpression` signature to use `Pattern`
@agcom
Refactored regex tests
9fbc52e 
Closes #46
@agcom agcom added the bug Something isn't working label Sep 19, 2020
@agcom agcom added this to the v1.1.0 milestone Sep 19, 2020
@agcom agcom self-assigned this Sep 19, 2020
@agcom agcom merged commit 0a3d2ff into main Sep 19, 2020
@agcom agcom deleted the regex branch September 19, 2020 15:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@agcom agcom

Labels

bug Something isn't working

Projects

None yet

Milestone

v0.3.0

Development

Successfully merging this pull request may close these issues.

1 participant

@agcom