Closed justin-tay closed 3 weeks ago
Attention: Patch coverage is 93.22034%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 78.86%. Comparing base (
48ca3c2
) to head (3e41dee
). Report is 21 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
...hema/regex/ECMAScriptRegularExpressionFactory.java | 76.92% | 1 Missing and 2 partials :warning: |
.../networknt/schema/regex/GraalJSContextFactory.java | 75.00% | 1 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This adds the optional usage of GraalJS as the regular expression evaluation engine. This makes it somewhat easier to verify that the Joni regular expression implementation is correct.
This also fixes the JoniRegularExpression implementation as the
Syntax.ECMAScript
wasn't accepting certain valid regular expressions by configuring a customSyntax
with additional options