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 const instead of let there.
*Anindya Kundu* On Sat, 9 May 2020 at 15:55, Vaibhav Aren <vaibhavda...@gmail.com> wrote: > Lets peer review these first and get them merged on priority. > > [1] https://github.com/sugarlabs/musicblocks/pull/2231 > [2] https://github.com/sugarlabs/musicblocks/pull/2226. > [3] https://github.com/sugarlabs/musicblocks/pull/2225 > [4] https://github.com/sugarlabs/musicblocks/pull/2208. > [5] https://github.com/sugarlabs/musicblocks/pull/2224. > [6] https://github.com/sugarlabs/musicblocks/pull/2201 > <https://github.com/sugarlabs/musicblocks/pull/2201/files>. > > I can see some of your reviews already. > -Vaibhav > _______________________________________________ > Sugar-devel mailing list > Sugar-devel@lists.sugarlabs.org > http://lists.sugarlabs.org/listinfo/sugar-devel >
_______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
