Install git in runtime container

[joeshannon]

This is required for installing python packages at runtime.

It was previously present in the non-slim python container used before but this was accidentally removed with the switch to the copier template.

Fixes #445.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.16%. Comparing base (0a634dc) to head (2385dd5).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #446   +/-   ##
=======================================
  Coverage   90.16%   90.16%           
=======================================
  Files          41       41           
  Lines        1708     1708           
=======================================
  Hits         1540     1540           
  Misses        168      168           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joeshannon]

I think this should work, I did look in the action logs though and it wasn't clear where git was being installed.

Also the apt in the container above has && rm -rf /var/lib/apt/lists/* at the end of it - do we really need this here too?

[joeshannon]

I think this should work, I did look in the action logs though and it wasn't clear where git was being installed.

It was being installed but I missed it before as it seemed to be building the runtime container in parallel with the developer container.

[joeshannon]

The copier template has && rm -rf /var/lib/apt/lists/* after the apt-get.

@coretl @gilesknap Is this some sort of best practice for using apt in Dockerfiles? Should I include it here too?

[coretl]

Yes, reading the docker best practices you get examples that look like that:
https://docs.docker.com/develop/develop-images/guidelines/#sort-multi-line-arguments

[joeshannon]

Thanks!