Opened 7 weeks ago

Last modified 5 days ago

#7344 reviewing change

Remove handling of "Fixes XXXX - ..." commit messages

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

https://codereview.adblockplus.org/30048555/

Description

Background

With #3681 we added support for commit messages like "Fixes XXXX - ..." which would cause the commit to be referenced in the respective issue on Trac, and the issue to be closed if set up accordingly (which it never was). However, since it was never was set up to actually close the issue, it was rarely used to reference issues on Trac, but it's now conflicting with the notation used to reference issues on GitLab, potentially causing comments about unrelated commits being posted to old issues on Trac.

What to change

Revert the changes implemented in #3681.

Change History (3)

comment:1 Changed 6 days ago by kvas

  • Owner set to kvas

comment:2 Changed 6 days ago by kvas

  • Priority changed from Unknown to P3
  • Ready set

comment:3 Changed 5 days ago by kvas

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.