Skip to content

Conversation

@sebastienbeau
Copy link
Member

@sebastienbeau sebastienbeau commented Feb 11, 2025

@hparfr if it's fine for you I will do it for the other version of the image

@@ -0,0 +1,16 @@
#!/bin/bash
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see no benefit for this. It will not work with docker exec for instance. So having to repeat the twice the args in the docker-compose is still needed.

# Install tooling for running test and coverage
ENV PIPX_HOME=/opt/pipx PIPX_BIN_DIR=/usr/local/bin
RUN --mount=type=cache,target=/root/.cache\
python3 -m pip install pipx &&\
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the value of using pipx inside the container ?

RUN --mount=type=cache,target=/root/.cache\
python3 -m pip install pipx &&\
pipx install manifestoo &&\
pipx install coverage
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ak_requirement instead or may be in the project directly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants