Feature Request - Add configuration to disable passing comments

[atavakoliyext]

I'd like to only have comments when there are violations, but it doesn't appear there's a way to disable the passing comments.

I've tried setting message.scorePassingComment= in the configuration, but this is equivalent to leaving the configuration out, causing it to fall back to the default of "Perfect!"

It would be great to have a configuration that allows these passing comments to be disabled entirely.

[SpOOnman]

Do I understand you correctly that if there are no comments, with message.scorePassingComment= instead of a "Perfect!" there should be no comment at all?

[atavakoliyext]

It does not necessarily have to be message.scorePassingComment= that does this, but yes; we'd like some configuration option that posts no comment when there are no violations.

[atavakoliyext]

@SpOOnman Is this something you would consider implementing?

To clarify the motivation in my team's case, our workflow uses Gerrit with the Checks plugin, with Sputnik being one of those checks. Because the plugin already surfaces results of each check in the Gerrit UI and in emails, having the additional message/email from Sputnik is unnecessary, particularly since there are no actions for the receiver to take when no violations are found.

Longer term, I hope Sputnik can integrate with Gerrit's "Findings" feature to report violations, but in the meantime, having a way to disable notification on violations would help greatly in reducing inbox/comment-thread noise in our reviews.

[SpOOnman]

Thank you for reporting. I'll implement that.

[atavakoliyext]

@SpOOnman have you had an opportunity to look into this request further?