<div dir="ltr"><div>I've reviewed [1], [2], and [3]. As, Aviral pointed out, 4 commits were drawn in from previous pulls too. There's only one tiny commit remaining. However, there's a suggestion to use<font face="monospace"> const </font>instead of<font face="monospace"> let </font>there.</div><div><br></div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font size="2" color="#666666"><i><span style="font-family:"arial narrow",sans-serif">Anindya Kundu</span></i></font></blockquote><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font size="2"><span style="font-family:"arial narrow",sans-serif"></span></font></blockquote></div></div></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 9 May 2020 at 15:55, Vaibhav Aren <<a href="mailto:vaibhavdaren@gmail.com">vaibhavdaren@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Lets peer review these first and get them merged on priority. <br><br>[1] <a href="https://github.com/sugarlabs/musicblocks/pull/2231" target="_blank">https://github.com/sugarlabs/musicblocks/pull/2231</a>  <br>[2] <a href="https://github.com/sugarlabs/musicblocks/pull/2226" target="_blank">https://github.com/sugarlabs/musicblocks/pull/2226</a>. <br>[3] <a href="https://github.com/sugarlabs/musicblocks/pull/2225" target="_blank">https://github.com/sugarlabs/musicblocks/pull/2225</a>  <br>[4] <a href="https://github.com/sugarlabs/musicblocks/pull/2208" target="_blank">https://github.com/sugarlabs/musicblocks/pull/2208</a>. <br>[5] <a href="https://github.com/sugarlabs/musicblocks/pull/2224" target="_blank">https://github.com/sugarlabs/musicblocks/pull/2224</a>. <br>[6] <a href="https://github.com/sugarlabs/musicblocks/pull/2201/files" target="_blank">https://github.com/sugarlabs/musicblocks/pull/2201</a>. <br><br>I can see some of your reviews already.<br>-Vaibhav</div>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</blockquote></div>