[BB-5005] Rework ticket injection behaviour
Currently, if crafty finds a ticket that fits the criteria for a sprint injection (moving a ticket into next sprint after the thursday of the current sprint), it will move the ticket to stretch goals and leave a comment about it being an injection.
as a sprint planning manager, I really like the comment with the warning about an injection (this helps me track tickets that are injected), but I don't like the auto move (this is prone to false positives, and makes more manual work to get it back into the sprint).
I propose that we change this behaviour:
- drop the automated ticket moving
- if a ticket is identified as a potential sprint injection, crafty should leave a comment like this on the ticket as a warning:
@epic-manager @sprint-planning-manager this ticket was added to the sprint at DATETIME and may be a sprint injection. Please check the ticket and move to stretch goals or a later sprint if it cannot be injected.
CC @nizarmah @x29a what do you think about this?
Edited by Piotr Surowiec