search

packit / tmt-plans

Shared library of TMT plans
MIT License
0 stars 3 forks source link

fix: rpminspect report #16

Closed LecrisUT closed 10 months ago

LecrisUT commented 10 months ago

Refined the reporting of rpminspect so that the plan does not fail on VERIFY results

Depends on: #15

lachmanfrantisek commented 10 months ago

@LecrisUT Can you please rebase the PR so it can be merged nicely?

LecrisUT commented 10 months ago

@LecrisUT Can you please rebase the PR so it can be merged nicely?

Weird, I thought that since the other commits are already in history it would only show the 2 extra commits, oh well, I rebased it quickly

lachmanfrantisek commented 10 months ago

Weird, I thought that since the other commits are already in history it would only show the 2 extra commits, oh well, I rebased it quickly

Thanks. Maybe because we use the merge commits?

LecrisUT commented 10 months ago

Weird, I thought that since the other commits are already in history it would only show the 2 extra commits, oh well, I rebased it quickly

Thanks. Maybe because we use the merge commits?

We already did, it's just Github fails to detect that :face_exhaling:. Anyway, no big deal as long as the button to rebase is available on the repo