RMI-PACTA / workflow.transition.monitor

This repository is used to create the docker image that runs the transition monitor platform.
Other
2 stars 0 forks source link

consider whether {pacta.portfolio.import} needs to be installed before {pacta.portfolio.analysis} #109

Closed cjyetman closed 1 year ago

cjyetman commented 1 year ago

related #82

Since {pacta.portfolio.import} is a dependency of {pacta.portfolio.analysis}, we should consider if we should change the ordering of package installs in the Dockerfile because the install of {pacta.portfolio.analysis} will cause an install of {pacta.portfolio.import} directly from GitHub rather than installing from the clone of the repo copied in, which could lead to version conflicts if we were intending to install a specific, non-main version of {pacta.portfolio.import}.

cjyetman commented 1 year ago

according to the pak::pkg_install() documentation:

So it seems that the order of installation does not matter because in either case we should end up with the {pacta.portfolio.import} version intended, because either...

@jdhoffa @AlexAxthelm id that how you understand it? do you agree? shall we close this then?

jdhoffa commented 1 year ago

I tend to agree with what you said here, yes!

jdhoffa commented 1 year ago

Happy to close.

AlexAxthelm commented 1 year ago

makes sense to me.