Opened 5 years ago

Closed 5 years ago

#4053 closed defect (bug) (fixed)

Remove `/committers` command in Slack

Reported by: desrosj's profile desrosj Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: Communication (Matrix, Slack, IRC) Keywords: has-patch


The /committers command in slack has long been superseded by the @ committers group. It was recently accidentally used and a few problems came to light.

  1. It results in double pings because @ committers and the individual Slack handles are included.
  2. The list is out of date.

Attachments (2)

4053.diff (2.4 KB) - added by desrosj 5 years ago.
4053.2.diff (3.2 KB) - added by desrosj 5 years ago.

Download all attachments as: .zip

Change History (7)

5 years ago

#1 @desrosj
5 years ago

4053.diff removes /committers and updates the get_committers() list to be what I believe is currently accurate.

5 years ago

#2 @desrosj
5 years ago

4053.2.diff also updates core.trac so that the new committers have the Core Committer label on their comments.

#3 @dd32
5 years ago

In 8068:

Slack: Remove the /committers slash command handler as it's no longer needed.

The Slack slash integration has also been disabled.

Props desrosj.
See #4053.

#4 @dd32
5 years ago

In 8069:

Slack: Update the list of Committers for slash command whitelisting purposes.

Props desrosj, dd32.
See #4053.

#5 @dd32
5 years ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 8070:

Trac: Update the list of Committers to add their proper titles on Trac.

Props desrosj, dd32.
Fixes #4053.

Note: See TracTickets for help on using tickets.