Replace Jackson 2 API with Jackson 3#684
Conversation
|
@uhafner If any interest |
uhafner
left a comment
There was a problem hiding this comment.
I finally managed it to move the baseline to a more modern Jenkins version...
Can't we simply ignore the exceptions now?
8b6f076 to
3035c24
Compare
It seems that it still makes sense to catch the exceptions. (Or we should delete the test? Throwing a runtime exception might help to find broken configurations) |
Ok I will check tomorrow. I think it doesn't harm to catch it and rethrow a different runtime exception |
This reverts commit 3035c24.
☀️ Quality MonitorTests Coverage for New Code 〰️ Line Coverage: 50.00% — 1 missed lines Coverage for Whole Project 〰️ Line Coverage: 76.24% — 727 missed lines Style Bugs Vulnerabilities🛡️ OWASP Dependency Check: 206 vulnerabilities — error: 18, high: 53, normal: 129, low: 6 Software Metrics 🌀 Cyclomatic Complexity: 1028 (total) 🚦 Quality GatesOverall Status: ❗ UNSTABLE✅ Passed Gates
❌ Failed Gates
Created by Quality Monitor v4.10.0 (#be1b382). More details are shown in the GitHub Checks Result. |
Jackson3 update
Testing done
mvn -DskipTests clean install
Submitter checklist