-
Notifications
You must be signed in to change notification settings - Fork 669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ticket Reference handling #1048 #1055
Conversation
paulsputer
commented
Apr 11, 2016
- Supports referencing:
- from other tickets via comments
- from commits on other branches
@gitblit could you cast an eye over this and share your thoughts? |
I will cast my eye. Trying to make time. |
Thoughts...
|
Thanks James, (1) will do, (2) very true, I tried to use the lowest level lozenge but still looks a bit too much, I'll make those fixes and post back. Are you happy with the temporary TicketLink and persisted Reference structures? I'm not sure if there's any cleaner alternative. |
0536531
to
2c91b98
Compare
+ Supports referencing: + Tickets from other tickets via comments + Tickets from commits on any branch + Common TicketLink class used for both commits and tickets + TicketLink is temporary and persisted to ticket as a Reference + Support deletion of ticket references + Rebasing patchsets/branches will generate new references + Deleting old patchsets/branches will remove the relevant references + Substantial testing of use cases + With and without patchsets, deleting, amending + BranchTicketService used during testing to allow end-to-end ref testing + Relocated common git helper functions to JGitUtils
5f4bfa3
to
c2188a8
Compare
ok @gitblit can you take another look at this and see if there's anything else we need to address on referencing in this PR? It's getting a bit larger than I was anticipating, then again half of it is testing the different use cases. |
+ Multiple commits in a single push