Jenkins 56333: pc-lint: categorize errors as errors and not warning (high)

[DaMutz]

No description provided.

[codecov]

Codecov Report

Merging #129 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #129   +/-   ##
=========================================
  Coverage     87.81%   87.81%           
  Complexity     1211     1211           
=========================================
  Files           157      157           
  Lines          3857     3857           
  Branches        425      425           
=========================================
  Hits           3387     3387           
  Misses          312      312           
  Partials        158      158

Impacted Files	Coverage Δ	Complexity Δ
...a/edu/hm/hafner/analysis/parser/MsBuildParser.java	80.7% <100%> (ø)	19 <0> (ø)	⬇️

---

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 4d30763...8ba2e35. Read the comment docs.

[codecov]

Codecov Report

Merging #129 into master will decrease coverage by 0.01%.
The diff coverage is 60%.

@@             Coverage Diff              @@
##             master     #129      +/-   ##
============================================
- Coverage     87.85%   87.84%   -0.02%     
+ Complexity     1211     1207       -4     
============================================
  Files           156      156              
  Lines          3855     3850       -5     
  Branches        425      424       -1     
============================================
- Hits           3387     3382       -5     
- Misses          310      311       +1     
+ Partials        158      157       -1

Impacted Files	Coverage Δ	Complexity Δ
src/main/java/edu/hm/hafner/analysis/Severity.java	96.29% <100%> (ø)	26 <0> (ø)	⬇️
...a/edu/hm/hafner/analysis/parser/MsBuildParser.java	78.84% <50%> (-1.86%)	15 <0> (-4)

---

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 d6e19f2...ed813ba. Read the comment docs.

[uhafner]

Would it make sense to add "note" to Severity.guessFromString as well?

[DaMutz]

Would it make sense to add "note" to Severity.guessFromString as well?

yes, it looks good with the 'note'. In general there are not many test cases for this parser so the only possibility to improve it to get more data or wait for issues in jenkins.

[uhafner]

Thanks!