https://pulumi.com logo
#general
Title
# general
c

cool-egg-852

10/07/2019, 6:02 PM
@colossal-beach-47527 Since I just saw your comment on the yarn/npm thing (I already replied), do you guys plan to port this logic over to other stuff such as the policy packs?
If it’s not ok to bring this up here, let me know. I figured it be ok given the announcement.
c

colossal-beach-47527

10/07/2019, 6:13 PM
I don’t believe we want the
pulumi
command-line client to do too much “magic” behind the scenes with regard to running
npm install
,
yarn install
, etc. Since there are plenty of corner cases we’d probably have trouble getting right. (e.g. if you are using a custom package manager.) So I think we’d want to leave “gathering runtime dependencies” to whatever script or CI/CD environment you are using to execute
pulumi
. (So that then you can confirm you have the right setup.) But what I think we can/should/will do is make sure that we do have a good escape hatch so you can tie custom steps into the scripts or containers that we publish. For example, the
entrypoint.sh
for the
pulumi/pulumi
Docker container (like in your PR). Or supporting a generic
if [ -f pulumi-pre-update.sh ]
or something. Make sense? This is a problem we want to make sure gets solved, but it probably makes more sense to solve it upstream.
c

cool-egg-852

10/07/2019, 6:13 PM
It already runs
npm install
though (hardcoded).
m

magnificent-leather-3361

10/07/2019, 6:27 PM
Hi @cool-egg-852! We do have an open issue to fix this for the policy packs! If you any input/suggestions, feel free to DM me and we can discuss more.
c

cool-egg-852

10/07/2019, 6:28 PM
Sounds good, thanks.