MrMustard icon indicating copy to clipboard operation
MrMustard copied to clipboard

Refactor devcontainer πŸ“¦

Open sduquemesa opened this issue 1 year ago β€’ 1 comments

Context: Doing some housekeeping here on the devcontainer side.

Description of the Change:

  • The previous build script was a bit cumbersome, here it has been simplified.
  • Julia installation now automatically detects architectures, meaning there is no need for passing the optional JULIA_INSTALL arg to docker build.
  • Only a single Dockerfile to create the container. This Dockerfile can also be used to create a completely isolated environment using docker (useful for testing, running simulations, etc.).
  • Ensures changes on the code are reflected on the interpreter as they happen. To do so the workspace folder points to /mrmustard where the source code resides.
  • Installs the jupyter extension automatically.

Benefits: Click and go πŸš€

Possible Drawbacks: None

sduquemesa avatar Jan 26 '24 22:01 sduquemesa

@ziofil, I've readded oh-my-zsh to the container as default terminal

sduquemesa avatar Jan 31 '24 20:01 sduquemesa

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.62%. Comparing base (a638bfb) to head (6460987).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #329   +/-   ##
========================================
  Coverage    87.62%   87.62%           
========================================
  Files           81       81           
  Lines         6158     6158           
========================================
  Hits          5396     5396           
  Misses         762      762           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update a638bfb...6460987. Read the comment docs.

codecov[bot] avatar May 29 '24 20:05 codecov[bot]