rootstock icon indicating copy to clipboard operation
rootstock copied to clipboard

Build docker image via GitHub Actions

Open dhimmel opened this issue 3 years ago • 4 comments

Very preliminary proof of concept. The idea is to have CI create a Docker image for the environment. This would have two potential benefits:

  1. improve build times for subsequent builds that do not modify the environment
  2. make the CI environment available for local usage, allowing users to return to any past environment

dhimmel avatar May 29 '21 22:05 dhimmel

AppVeyor build 1.0.238 for commit 205a07308fdd0170db9d17cf5ec3a56636bba474 is now complete.

Found 52 potential spelling error(s). Preview:content/02.delete-me.md:44:adipiscing content/02.delete-me.md:44:aliqua content/02.delete-me.md:44:amet content/02.delete-me.md:44:consectetur content/02.delete-me.md:44:dolore content/02.delete-me.md:44:eiusmod content/02.delete-me.md:44:elit content/02.delete-me.md:44:incididunt content/02.delete-me.md:44:ipsum content/02.delete-me.md:44:labore content/02.delete-me.md:44:Lorem content/02.delete-me.md:44:magna content/02...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot avatar May 29 '21 23:05 AppVeyorBot

AppVeyor build 1.0.239 for commit df9ac42a36c3ff05fbb89af8d00e04994c0c913b is now complete.

Found 52 potential spelling error(s). Preview:content/02.delete-me.md:44:adipiscing content/02.delete-me.md:44:aliqua content/02.delete-me.md:44:amet content/02.delete-me.md:44:consectetur content/02.delete-me.md:44:dolore content/02.delete-me.md:44:eiusmod content/02.delete-me.md:44:elit content/02.delete-me.md:44:incididunt content/02.delete-me.md:44:ipsum content/02.delete-me.md:44:labore content/02.delete-me.md:44:Lorem content/02.delete-me.md:44:magna content/02...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot avatar May 29 '21 23:05 AppVeyorBot

AppVeyor build 1.0.240 for commit 1eb3194a80d92022601c93ce1d444c2bdbfb09d5 is now complete.

Found 52 potential spelling error(s). Preview:content/02.delete-me.md:44:adipiscing content/02.delete-me.md:44:aliqua content/02.delete-me.md:44:amet content/02.delete-me.md:44:consectetur content/02.delete-me.md:44:dolore content/02.delete-me.md:44:eiusmod content/02.delete-me.md:44:elit content/02.delete-me.md:44:incididunt content/02.delete-me.md:44:ipsum content/02.delete-me.md:44:labore content/02.delete-me.md:44:Lorem content/02.delete-me.md:44:magna content/02...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot avatar May 29 '21 23:05 AppVeyorBot

Great idea. Both of those benefits sound good to me and should be achievable.

How much of the CI environment will eventually be added into the Docker image? Initially it has the conda environment. Will spellcheck, PDF generation, or anything else be added later? In particular, I see that the current version uses weasyprint. I don't know much about running Docker (for athenapdf) inside Docker but presume that isn't recommended.

If you don't want to use the older style of conda activation (add to PATH and source activate), we have used the template from this blog post in other projects. It sets the ENTRYPOINT to use conda run to run commands inside the desired environment.

agitter avatar May 30 '21 21:05 agitter