----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110287/#review32019 -----------------------------------------------------------
Ship it! whoops, hit the wrong button there. wanted to say: there's just the one small issue, but this looks god and should go in given the direction of npm. - Aaron J. Seigo On May 4, 2013, 10:20 a.m., Giorgos Tsiapaliokas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110287/ > ----------------------------------------------------------- > > (Updated May 4, 2013, 10:20 a.m.) > > > Review request for Bodega. > > > Description > ------- > > Hello, > > this patch does > > 1. it requires 3 parameters now instead of 1. > The new parameters are the config.json filepath, the sql dir and the sql > command. > 2. it removes the npm scripts functionality > > check this issue > https://github.com/isaacs/npm/issues/3370#issuecomment-16892283 > > 3. it adds the sql functionality into the makefile. > > > Diffs > ----- > > server/Makefile a35b309 > server/package.json 3ee556d > sql/do_sql 93bf764 > > Diff: http://git.reviewboard.kde.org/r/110287/diff/ > > > Testing > ------- > > > Thanks, > > Giorgos Tsiapaliokas > >
_______________________________________________ Active mailing list Active@kde.org https://mail.kde.org/mailman/listinfo/active