Page MenuHomeSoftware Heritage

Push diff notifications for review on irc channels
Closed, MigratedEdits Locked

Description

It has been suggested more than once.
Recently, through our code review discussion on the devel mailing list, so we might want to give it a try.

Something like, diffs with group reviewers:

  • #sysadm, the url should be pushed to #swh-sysadm irc
  • #reviewers, the url should be pushed to #swh-devel irc channel

Event Timeline

ardumont renamed this task from Push diff for review on irc channels to Push diff notifications for review on irc channels.May 17 2021, 10:56 AM
ardumont triaged this task as Normal priority.
ardumont created this task.

I have concerns about adding lots of noise to the main devel irc channel, so I would suggest having these in a separate channel first.

I think the sysadm channel is already lost for noise, so we can have the notifs there directly :(

The Phabricator limnoria module we're using has some provisions for reading the Phabricator activity feed, I turned it off because it was somewhat buggy, but that was years ago...