Mounil Shah

@zulipbot add "PR needs review"

@zulipbot remove "PR awaiting update" @zulipbot add "PR needs review"

> The commit flow is not what I would expect. It currently represents the way in which you've done the development rather than adding in stages to illustrate the changes....

> it does not display a success message in one case. I'll add it, thanks for pointing that out.

> @mounilKshah Just a note that generally it's fine (at least in Zulip) to force push a rebased version to a PR branch (where we also avoid merge commits). It's...

@zulipbot remove "PR awaiting update" @zulipbot add "PR needs review"

@imnitishng has this issue been resolved or is it fine if I take it up?

@zulipbot claim

As my PR #1146 has been approved, the issue which was earlier assigned to me #1095 is fixed. Hence, I am beginning work on this issue. I hope that's fine.

@zulipbot claim