search

dwyl / dwylbot

:robot: Automating our GitHub Workflow to improve team communication/collaboration and reduce tedious repetition!
28 stars 7 forks source link

Amend markdown error on time estimation dwylbot comment #114

Closed Cleop closed 7 years ago

Cleop commented 7 years ago

The dwylbot comment for #61, an issue that is in-progress but with no time estimation is currently: image

The backtick on the last in-progress is too far and has covered the word label, move this for consistency so it looks the same as the first one.

SimonLab commented 7 years ago

@Cleop the issue is in "awaiting-review" but it doesn't seem to have a PR open linked to the issue, is this right?

Cleop commented 7 years ago

Sorry, I hadn't pushed up but had put awaiting review because it was no longer in-progress. Try now 😊

SimonLab commented 7 years ago

Good user testing, I think if the issue is no longer in progress we should just remove the "in-progress" label and only add the "awaiting-review" when the PR is ready. But maybe my definition of "awaiting-review" on issue is not right, what do you think?

Cleop commented 7 years ago

I agree that my use of the awaiting-review label in that context is perhaps a little misleading. However, the reason I do it is because I don't want to ruin the time estimates recorded using in-progress and if I leave the issue with no progress labels at all, I find it more work to go back in and assign awaiting-review manually later. Whereas if I remove in-progress and switch straight to awaiting-review then I've smoothed that action into one visit to the issue (which if you have a lot of issues can be time consuming). I know you can tick multiple issues and change their labels at a time but then I find it hard remembering / altering the correct labels and not missing any out. Obviously you can look at the issues mentioned by number in your PR, but if dwylbot could help me not have to open those tabs again and change them, I would find this very useful!

ghost commented 7 years ago

Closing as this was merged