revisions skrz cp.execSync(cd && git) nejedou, kdyz nb-builder je jako dependency

[ptica]

chtel jsem se vyhnout execu, tak jsem si udelal malickeho consumera z fronty v redisu:
https://github.com/ptica/nb-builder-queue/blob/master/worker.js#L13

lib/revision.js predpoklada, ze je nb-builder checkoutnutej z gitu,
ale https://github.com/ptica/nb-builder-queue/blob/master/package.json#L12 mi to da bez .git

mam zaklad i pro web administraci nbooku do novabeseda.cz, tam by se zadavali jako url na github repo s dialogem na oauth token skrz browser, ale pres github api jsem zatim tahal tarball,
takze i v pripade zdroje nbooku by se mi trochu vic hodila jina cesta pro zjisteni revizi

v tom tarballu z githubu je myslim podadresar s hashem verze, u nb-builderu je myslim tohle info v yarn.lock

exec je ale taky moznost ;)

zatim jsem to poeditoval rucne

const build = srcDir => ({
  bookRevision: getRevision(srcDir),
  bookDate: new Date(getDate(srcDir)).toISOString(),
  //builderRevision: getRevision('.'),
  builderRevision: '02c997d560797daaf6b111181a21daf165099a5c',
  //builderDate: new Date(getDate('.')).toISOString(),
  builderDate: new Date('Sat Jul 1 22:03:37 2017 +0200').toISOString(),
  compiledDate: new Date().toISOString(),
});

[jan-martinek]

co jsem koukal, tak v tarballu to info neni, ale kdyz jdes pres API, dostanes hash v response.

u samotne knizky beru primou praci s obsazenym git repozitarem jako zasadni — a exec byl nejjednodussi na napsani. koukl jsem okolo az ted, tak vidim, ze by to slo prepsat za uziti nodegit.

u toho builderu jsem to zjistovani verzi doplnil pomerne pozde a nemam to nijak zvlast domyslene — kdyby byl nalistovany jako dependency primo u knizky, nebylo by nutne verzi resit — ale zase by pak bylo nutne to spoustet primo z te instance dostupne v knizce. to by resilo problem s verzovanim builderu i nad ramec teto diskuse (builder i kod knizky se jeste dost dramaticky meni), ale nevim, jestli to prilis nerozbije workflow na serveru — je to pro tebe ok? jinak tam klidne prozatim hod nejaky workaround.