Skip to content
This repository was archived by the owner on Aug 15, 2024. It is now read-only.

Remove Unecessary COPY Operation from Dockerfile #494

Open
thefringeninja wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Remove Unecessary COPY Operation from Dockerfile #494

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
38d710e
remove unecessary COPY operation
thefringeninja Jan 20, 2021
39f9a3c
don't continue on error
thefringeninja Jan 25, 2021
1d1286a
don't restore build script dependencies
thefringeninja Jan 25, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,14 @@ WORKDIR /repo
# https://code-maze.com/aspnetcore-app-dockerfiles/
COPY ./*.sln ./

COPY ./build/ ./build/

COPY ./src/*/*.csproj ./src/
RUN for file in $(ls src/*.csproj); do mkdir -p ./${file%.*}/ && mv $file ./${file%.*}/; done

COPY ./tests/*/*.csproj ./tests/
RUN for file in $(ls tests/*.csproj); do mkdir -p ./${file%.*}/ && mv $file ./${file%.*}/; done

RUN dotnet restore
RUN find src -name "*.csproj" | xargs -n1 dotnet restore && \
find tests -name "*.csproj" | xargs -n1 dotnet restore

COPY ./assets ./assets/

Expand Down
3 changes: 2 additions & 1 deletion build.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
#!/usr/bin/env bash
set -eo pipefail

docker build --tag sss-build .
docker run --rm --name sss-build \
-v /var/run/docker.sock:/var/run/docker.sock \
-v $PWD/artifacts:/repo/artifacts \
-v $PWD/.git:/repo/.git \
--network host \
-e FEEDZ_SSS_API_KEY=$FEEDZ_SSS_API_KEY \
-e FEEDZ_SSS_API_KEY=${FEEDZ_SSS_API_KEY} \
sss-build \
dotnet run -p /repo/build/build.csproj -- "$@"