<div dir="ltr">Hi.<div><br></div><div>I think that's a good idea. This will greatly facilitate the review of pull requests and the development of the GSoC project.<br></div><div><br></div><div>I created a new branch called gsoc-js-refactoring [1] and changed the base branch of the first PR to that branch [2].<br></div><div><br></div><div>[1] <a href="https://github.com/phpmyadmin/phpmyadmin/tree/gsoc-js-refactoring">https://github.com/phpmyadmin/phpmyadmin/tree/gsoc-js-refactoring</a></div><div>[2] <a href="https://github.com/phpmyadmin/phpmyadmin/pull/14464">https://github.com/phpmyadmin/phpmyadmin/pull/14464</a></div><div><br></div><div>Regards,</div><div>Maurício</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 6, 2018 at 11:01 AM Piyush Vijay <<a href="mailto:piyushvijay.1997@gmail.com">piyushvijay.1997@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><img width="0" height="0" class="m_-7510587812299578347mailtrack-img" alt="" style="display:flex" src="https://mailtrack.io/trace/mail/a3af2bcfe00841a9e2e888f5f403169ead7936f6.png?u=1589778"><div></div><div></div><div></div><div></div>Hi,<div><br></div><div>Me and Devan were discussing about the current pr 'gsoc' which I have made for the modular javascript code but the pr has lots of unused files which I have created for making sure that existing code is not breaking and contains lot of commented out code.<br><br>So we have decided to create a new branch in main repo of phpMyAdmin and multiple pr's from gsoc branch with small changes which can be reviewed and then merged into the newly created branch and we'll then keep merging this new branch into master after review and testing.</div><div><br></div><div>So I am closing this gsoc pr and will make multiple small pr's from this branch by cherry-picking the commits.<br><br>Any suggestions regarding this would be welcome.</div><div><br></div><div>Regards</div><div>Piyush Vijay</div></div>
</blockquote></div>