Add signature verification status icon next to each tarball in staging PR body#250
Conversation
|
Testing in https://github.com/EESSI/staging_bundles/pull/44. |
trz42
left a comment
There was a problem hiding this comment.
Looks good to me (only a very minor suggestion). I guess the additional payload_verified is added to the config setting
task_summary_payload_template.
I noted that we have no example configuration file (similar to automated_ingestion.cfg.example) and also the README.md is outdated. I think these could be handled in a follow-up PR too.
I've opened an issue for the example configuration file: #251. The README is more or less mentioned in #245. |
trz42
left a comment
There was a problem hiding this comment.
Thanks for the update and new issue. Looks good to me.
Not tested yet. Partially solves #244, but doesn't show yet which namespace successfully verified the signature. That's a bit more tricky, can be done in a follow-up PR.