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

Implement support for undefined values in JSONPath #88

Merged
merged 5 commits into from
Apr 11, 2022

Conversation

vassilvk
Copy link
Member

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #88 (448386e) into master (510da07) will increase coverage by 0.09%.
The diff coverage is 78.94%.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   78.64%   78.74%   +0.09%     
==========================================
  Files           8       13       +5     
  Lines         459      875     +416     
==========================================
+ Hits          361      689     +328     
- Misses         70      127      +57     
- Partials       28       59      +31     
Impacted Files Coverage Δ
jsonpath/jsonpath.go 63.63% <63.63%> (ø)
jsonpath/placeholder.go 68.04% <68.04%> (ø)
jsonpath/selector.go 79.33% <79.33%> (ø)
jsonpath/parse.go 81.88% <81.88%> (ø)
jsonpath/path.go 93.87% <93.87%> (ø)
core/wire_gen.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e46e7a5...448386e. Read the comment docs.

@vassilvk
Copy link
Member Author

Closes #87

@vassilvk vassilvk merged commit f97f428 into master Apr 11, 2022
@vassilvk vassilvk deleted the 87-extend-jsonpath-with-undefined branch April 11, 2022 21:12
@vassilvk vassilvk changed the title Implementation of support for undefined in JSONPath Implement support for undefined values in JSONPath Apr 11, 2022
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.

2 participants