support invoking node and npm

[JanTvrdik]

No description provided.

[Mikulas]

Nevím něco mi na tom nesedí, moc se mi nelíbí ten whitelist. Co když chci použít třeba yarn? Na tohle použití to nebylo myšlené, to už rovnou může člověk spustit třeba container s node:8.0.

Za mě jsou lepší řešení 2:

• mango v1.7.0 build & mango v1.7.0 -- npm install
• mango v1.7.0 mango build & mango v1.7.0 npm install

[enzy]

Držel bych to KISS, npm se dá spouštět z node image přímo, ne?

[JanTvrdik]

Na tohle použití to nebylo myšlené, to už rovnou může člověk spustit třeba container s node:8.0

Jenomže ty potřebuješ identický node a npm jako je v tom mango-cli containeru, protože jinak se ti může chovat jinak, než na CI.

mango v1.7.0 -- npm install

Jo, to je asi taky v pohodě, ale přijde mi to jako zbytečně složitá syntaxe.

[JanTvrdik]

Jinak mě už mango v1.7.0 npm install funguje. Jestli to bude fungovat vám je mi vlastně dost jedno. Klidně si ručně zjišťujte správnou verzi npm a nodu.

[enzy]

Jenomže ty potřebuješ identický node a npm jako je v tom mango-cli containeru, protože jinak se ti může chovat jinak, než na CI.

Není pravda, používaný npm install se chová vždy dle specifikace v package.json, která je zpětně kompatibilní. (v kontextu závislostí statického projektu)

Validní by to bylo, kdyby se npm install spouštěl pro přímé závislostí mango-cli, tam verze nodejs musí být stejná (kvůli nativním/kompilovaným modulům). Ale to v tomto případě neděláme.

Zároveň v nodejs aplikacích typicky chceš mít kontrolu nad verzí runtime a neomezovat se nad tou, která je v mango-cli image. V tomto případě dokonce musejí být nezávislé vzhledem k četnosti bezpečnostních aktualizací. (což toto řešení ale neomezuje)