Skip to content

Origin/vuln 2228#4

Open
carakanz wants to merge 1 commit into
developfrom
origin/VULN-2228
Open

Origin/vuln 2228#4
carakanz wants to merge 1 commit into
developfrom
origin/VULN-2228

Conversation

@carakanz
Copy link
Copy Markdown

Closes VULN-2228: check that expressions is not empty.

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 29, 2022

Codecov Report

Base: 98.73% // Head: 98.39% // Decreases project coverage by -0.34% ⚠️

Coverage data is based on head (89e2f2e) compared to base (56465d5).
Patch coverage: 100.00% of modified lines in pull request are covered.

❗ Current head 89e2f2e differs from pull request most recent head 01eb695. Consider uploading reports for the commit 01eb695 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop       #4      +/-   ##
===========================================
- Coverage    98.73%   98.39%   -0.35%     
===========================================
  Files           11       11              
  Lines          791      809      +18     
===========================================
+ Hits           781      796      +15     
- Misses          10       13       +3     
Impacted Files Coverage Δ
src/Parser.cpp 100.00% <100.00%> (ø)
src/Expander.cpp 97.14% <0.00%> (-1.86%) ⬇️
src/Variable.cpp 98.79% <0.00%> (-1.21%) ⬇️
include/uri-template/Template.h 100.00% <0.00%> (ø)
include/uri-template/Variable.h 100.00% <0.00%> (ø)
src/Matcher.cpp 96.25% <0.00%> (+0.07%) ⬆️

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@carakanz carakanz force-pushed the origin/VULN-2228 branch 2 times, most recently from 89e2f2e to 01eb695 Compare November 29, 2022 08:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant