Right, we discussed this a while back and it's working well.
On 19 Oct 2010, at 06:37, Stuart Douglas wrote:
I have been marking it as resolved when I add the pull request URL,
and Pete has been closing them after the changes have been merged.
This seems to work fairly well. I don't think we need to go 'Pull
Request Available' -> 'Resolved' -> 'Closed', as you will
want to close the issue after the code has been accepted into trunk.
On Tue, Oct 19, 2010 at 4:23 PM, Dan Allen <dan.j.allen(a)gmail.com> wrote:
> When a contributor has submitted a pull request as a candidate resolution to
> an issue, what issue state should the issue be put into?
> I'd like to suggest that we introduce a new state that indicates a pull
> request is pending and a review and, if accepted, merge is necessary. This
> state could be interpreted as a soft "Resolved".
> Dan Allen
> Principal Software Engineer, Red Hat | Author of Seam in Action
> Registered Linux User #231597
> seam-dev mailing list
weld-dev mailing list