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

[vividus] Introduce escapeJSON expression #3057

Merged
merged 1 commit into from
Aug 15, 2022

Conversation

valfirst
Copy link
Collaborator

Closes #3056

@codecov
Copy link

codecov bot commented Aug 12, 2022

Codecov Report

Merging #3057 (e893fa2) into master (f3101cf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3057   +/-   ##
=========================================
  Coverage     96.84%   96.84%           
  Complexity     5868     5868           
=========================================
  Files           825      825           
  Lines         16668    16668           
  Branches       1077     1077           
=========================================
  Hits          16142    16142           
  Misses          414      414           
  Partials        112      112           
Impacted Files Coverage Δ
...vividus/expression/StringsExpressionProcessor.java 93.75% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@valfirst valfirst force-pushed the introduce-escapejson-expression branch 2 times, most recently from 3c127cf to 4fa15a5 Compare August 12, 2022 14:20
@valfirst valfirst force-pushed the introduce-escapejson-expression branch from 4fa15a5 to e893fa2 Compare August 12, 2022 15:11
@sonarcloud
Copy link

sonarcloud bot commented Aug 12, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@valfirst valfirst merged commit 6343123 into master Aug 15, 2022
@valfirst valfirst deleted the introduce-escapejson-expression branch August 15, 2022 08:52
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.

Implement expression to escape JSON
2 participants