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

[Merged by Bors] - [regexp] new tests for unicode flag #2656

Closed
wants to merge 1 commit into from

Conversation

selfisekai
Copy link
Contributor

the flag got introduced in regress 0.5.0

This Pull Request fixes/closes: n/a.

It changes the following:

@jedel1043 jedel1043 added test Issues and PRs related to the tests. Internal Category for changelog labels Mar 13, 2023
@jedel1043 jedel1043 added this to the v0.17.0 milestone Mar 13, 2023
@codecov
Copy link

codecov bot commented Mar 13, 2023

Codecov Report

Merging #2656 (22ac3f4) into main (e4365b5) will increase coverage by 0.38%.
The diff coverage is n/a.

❗ Current head 22ac3f4 differs from pull request most recent head 97ea03d. Consider uploading reports for the commit 97ea03d to get more accurate results

@@            Coverage Diff             @@
##             main    #2656      +/-   ##
==========================================
+ Coverage   49.20%   49.59%   +0.38%     
==========================================
  Files         390      383       -7     
  Lines       39562    39228     -334     
==========================================
- Hits        19467    19454      -13     
+ Misses      20095    19774     -321     

see 17 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@raskad raskad left a comment

Choose a reason for hiding this comment

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

Nice addition. Only needs a cargo fmt and then we are good.

Thanks for the contribution :)

the flag got introduced in regress 0.5.0
Copy link
Member

@jedel1043 jedel1043 left a comment

Choose a reason for hiding this comment

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

Thanks!

@raskad
Copy link
Member

raskad commented Mar 14, 2023

bors r+

bors bot pushed a commit that referenced this pull request Mar 14, 2023
the flag got introduced in regress 0.5.0

<!---
Thank you for contributing to Boa! Please fill out the template below, and remove or add any
information as you feel necessary.
--->

This Pull Request fixes/closes: n/a.

It changes the following:

- add test for unicode flag
- add test for native error on failing to parse (got removed in #2651 because the previous example parsing correctly now)


Co-authored-by: Lauren N. Liberda <lauren@selfisekai.rocks>
@bors
Copy link

bors bot commented Mar 14, 2023

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title [regexp] new tests for unicode flag [Merged by Bors] - [regexp] new tests for unicode flag Mar 14, 2023
@bors bors bot closed this Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Category for changelog test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants