New "Pending PR Merge" column on waffle.io

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

New "Pending PR Merge" column on waffle.io

jewzaam
Administrator
For https://waffle.io/lightblue-platform/lightblue added new column "Pending PR Merge".  The intent is to put any issues that are linked to a PR into this to get them out of "in progress".  Makes it easier to see what is actually being worked on.  A few alternatives:
1) just close the issue when the PR is made
2) simply tag the issue, meaning it stays "In Progress"

Thoughts?
Reply | Threaded
Open this post in threaded view
|

Re: New "Pending PR Merge" column on waffle.io

lcestari
I thought that issues which the PR are pending to be merged would be on the ready for review column with its  PR.

So, if I solved an issue for instance, I move it to this new column and the PR for ready for review, right?
Reply | Threaded
Open this post in threaded view
|

Re: New "Pending PR Merge" column on waffle.io

jewzaam
Administrator

We could do that but as a reviewer you have to know not to directly review the issue. If we assume all issues are closed with a PR that is OK. This means changing how new association changes are pushed as an example.

On Sep 5, 2014 5:29 PM, "lcestari [via lightblue-dev]" <[hidden email]> wrote:
I thought that issues which the PR are pending to be merged would be on the ready for review column with its  PR.

So, if I solved an issue for instance, I move it to this new column and the PR for ready for review, right?


If you reply to this email, your message will be added to the discussion below:
http://dev.forum.lightblue.io/New-Pending-PR-Merge-column-on-waffle-io-tp138p141.html
To start a new topic under lightblue-dev, email [hidden email]
To unsubscribe from lightblue-dev, click here.
NAML