Opened on 05/05/2017 at 08:43:42 AM

Closed on 05/05/2017 at 02:52:30 PM

#5212 closed change (fixed)

IRC hook should report bookmarked branches

Reported by: trev Assignee: trev
Priority: P3 Milestone:
Module: Sitescripts Keywords:
Cc: kvas Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29430555/

Description

Background

Our Mercurial hook sending commit messages to IRC reports the branch that the commit happened on. It only reports Mercurial branches however - this is not too useful given that we mainly use bookmarked branches.

What to change

Send bookmarks for the branch to the IRC bot. Only use the Mercurial branch as fallback or if it isn't the default branch.

Attachments (0)

Change History (4)

comment:1 Changed on 05/05/2017 at 08:45:01 AM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed on 05/05/2017 at 08:56:54 AM by kvas

  • Priority changed from Unknown to P3
  • Ready set

comment:3 Changed on 05/05/2017 at 02:49:32 PM by abpbot

A commit referencing this issue has landed:
Issue 5212 - IRC hook should report bookmarked branches

comment:4 Changed on 05/05/2017 at 02:52:30 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
 
Note: See TracTickets for help on using tickets.