<div dir="ltr">Hi,<div><br></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 30, 2016 at 3:46 PM 殷啟聰 <<a href="mailto:seamlikok@gmail.com">seamlikok@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Chirayu,<br>
<br>
It is nice to enable code review for Debian packaging. But there are<br>
one thing we need to be wary of:<br>
<br>
The debian/changelog file is IMO the most likely place to cause a<br>
merge conflict. If one's changes are going through a code review, we<br>
can avoid touching debian/changelog and update it after the changes<br>
are pushed to the master repo.<br></blockquote><br class="inbox-inbox-Apple-interchange-newline"><div><span style="line-height:1.5">I agree with you here, I had some trouble trying to manage that file when creating the android-jar branch due to conflicts on every single commit.<br></span><br>Could have a page on the wiki under AndroidTools for some code review practices to follow for our team / repos in general. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Kai-Chung Yan<br>
<br>
2016-06-07 20:14 GMT+08:00 Chirayu Desai <<a href="mailto:chirayudesai1@gmail.com" target="_blank">chirayudesai1@gmail.com</a>>:<br>
> Hello everyone,<br>
><br>
> This topic was recently brought up on IRC (#debian-mobile), and a good<br>
> easy to setup workflow was suggested by Hans [1].<br>
> We also discussed using gerrit [2] (what upstream uses), however it's<br>
> setup and maintenance won't be an easy task, so it seems best to look<br>
> for an alternative workflow for now.<br>
><br>
> I have went ahead and forked all repos under android-tools on<br>
> <a href="http://git.debian.org" rel="noreferrer" target="_blank">git.debian.org</a> to my personal gitlab account [3]. They follow the<br>
> naming scheme debian_android-tools_<actual name><br>
> I have pushed all branches from <a href="http://git.debian.org" rel="noreferrer" target="_blank">git.debian.org</a> under alioth/, and have<br>
> also pushed the tags as-is.<br>
> I plan to push any changes I do here, and then ask for a review before<br>
> it gets pushed to <a href="http://git.debian.org" rel="noreferrer" target="_blank">git.debian.org</a>.<br>
><br>
> To sum this up, the workflow can be:<br>
> developer A pushes to his own fork on github/gitlab/etc<br>
> developers B, C, & so on have each other's forks as remotes along with upstream.<br>
> another developer (not A) can review the patch and then push it to<br>
> upstream if it looks good.<br>
><br>
> Alternatively, it can be used for staging / collaboration apart from<br>
> just code reviews.<br>
><br>
> Regards,<br>
> Chirayu Desai<br>
><br>
> [1]: <a href="https://guardianproject.info/2013/11/21/a-tag-team-git-workflow-that-incorporates-auditing/" rel="noreferrer" target="_blank">https://guardianproject.info/2013/11/21/a-tag-team-git-workflow-that-incorporates-auditing/</a><br>
> [2]: <a href="https://gerrit-review.googlesource.com/Documentation/intro-quick.html" rel="noreferrer" target="_blank">https://gerrit-review.googlesource.com/Documentation/intro-quick.html</a><br>
> [3]: <a href="https://gitlab.com/dashboard/projects" rel="noreferrer" target="_blank">https://gitlab.com/dashboard/projects</a><br>
><br>
> _______________________________________________<br>
> Android-tools-devel mailing list<br>
> <a href="mailto:Android-tools-devel@lists.alioth.debian.org" target="_blank">Android-tools-devel@lists.alioth.debian.org</a><br>
> <a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/android-tools-devel</a><br>
<br>
<br>
<br>
--<br>
/*<br>
* 殷啟聰 | Kai-Chung Yan<br>
* 一生只向真理與妻子低頭<br>
* Undergraduate student in National Taichung University of Education<br>
* LinkedIn: <<a href="https://linkedin.com/in/seamlik" rel="noreferrer" target="_blank">https://linkedin.com/in/seamlik</a>><br>
* Blog: <<a href="http://seamlik.logdown.com" rel="noreferrer" target="_blank">http://seamlik.logdown.com</a>><br>
*/<br>
</blockquote></div></div>