Open j-i-l opened 1 week ago
We are missing a development pipeline that allows us to run checks before merging.
> Looks OK, I think.
I'm used to define env variables in a docker with ENV PACKAGE_NAME=... but I do not see why running a classical declaration (i.e. RUN PACKAGE_NAME=...) should not work as well.
I'm used to define env variables in a docker with
ENV PACKAGE_NAME=... but I do not see why running a classical declaration (i.e. RUN PACKAGE_NAME=...) should not work as well.
ENV PACKAGE_NAME=...
RUN PACKAGE_NAME=...
https://github.com/furrer-lab/r-containers/actions/runs/9663922149/job/26657338485#step:6:36217
it wasn't OK it seems :see_no_evil:
We should add a dev pipeline that builds the containers on pull requests... I'll add this!
Originally posted by @j-i-l in https://github.com/furrer-lab/r-containers/issues/39#issuecomment-2189720024
We are missing a development pipeline that allows us to run checks before merging.
https://github.com/furrer-lab/r-containers/actions/runs/9663922149/job/26657338485#step:6:36217
it wasn't OK it seems :see_no_evil:
We should add a dev pipeline that builds the containers on pull requests... I'll add this!
Originally posted by @j-i-l in https://github.com/furrer-lab/r-containers/issues/39#issuecomment-2189720024