[Gluster-devel] Pull Request review workflow

[Ravishankar N]

On 15/10/20 4:36 pm, Sheetal Pamecha wrote:
>
> +1
> Just a note to the maintainers who are merging PRs to have patience 
> and check the commit message when there are more than 1 commits in PR.

Makes sense.

>
>         Another thing to consider is that rfc.sh script always does a
>         rebase before pushing changes. This rewrites history and
>         changes all commits of a PR. I think we shouldn't do a rebase
>         in rfc.sh. Only if there are conflicts, I would do a manual
>         rebase and push the changes.
>
>
>
I think we would also need to rebase if say some .t failure was fixed 
and we need to submit the PR on top of that, unless "run regression" 
always applies your PR on the latest HEAD in the concerned branch and 
triggers the regression.


>
> Actually True, Since the migration to github. I have not been using 
> ./rfc.sh and For me it's easier and cleaner.


Me as well :)

-Ravi
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gluster.org/pipermail/gluster-devel/attachments/20201015/c30709e8/attachment.html>