inet icon indicating copy to clipboard operation
inet copied to clipboard

[Request] Use encrypted email in Travis-CI notification to avoid spam from forks

Open screw opened this issue 6 years ago • 5 comments

Would you consider to use encrypted email in Travis-CI settings for notifications to avoid getting spam from forked repositories?

Reference: https://github.com/travis-ci/travis-ci/issues/5063#issue-115477841 https://github.com/facebook/flow/pull/1822

screw avatar Jan 06 '20 16:01 screw

Nice find, thanks! A bit hacky, but if it works... Then again, our Travis config is exceedingly cursed already.

torokati44 avatar Jan 06 '20 17:01 torokati44

Let's wait for an e-mail notification to arrive before closing this issue (this will most likely only happen next when some tests fail), just to confirm that decryption works, and I haven't made a mistake in the glorious YAML syntax.

torokati44 avatar Jan 06 '20 17:01 torokati44

This change has been in the mainline master for quite some time now, and also in @screw's fork a couple times, but not in all branches yet. If a build is triggered in a fork, on a branch/tag/hash that does not include this change, we will still get the notification about it.

torokati44 avatar Aug 10 '20 09:08 torokati44

Did I do that again?!?

screw avatar Aug 10 '20 09:08 screw

Yep, a couple times, last week, on the topic/ipsec branch. No problem though! :)

torokati44 avatar Aug 10 '20 09:08 torokati44

Can we close this issue?

levy avatar Feb 22 '23 08:02 levy

Yes, as we don't even use Travis CI for INET anymore.

torokati44 avatar Feb 22 '23 08:02 torokati44