search

jenkinsci / peass-ci-plugin

Jenkins plugin for peass to support performance measurement in CI
https://plugins.jenkins.io/peass-ci/
GNU Affero General Public License v3.0
4 stars 13 forks source link

Set buildstate unstable if json-file is not found in RCAVisualizer #231

Closed mawHBT closed 1 year ago

mawHBT commented 1 year ago

I'm not sure how to handle this. If this is an error, shouldn't buildstate be unstable? If it's not a hard error, message should maybe just be a warning?

DaGeRe commented 1 year ago

Thanks for the PR, this looks fine and should definitely be this way. Could you also add a unit test for this?