Skip to content

Commit a70c370

Browse files
committed
chore: apply suggestions
Applying simplifications and fixed workflow name as per review suggestions. Signed-off-by: Tim deBoer <git@tdeboer.ca>
1 parent ea78667 commit a70c370

File tree

3 files changed

+7
-12
lines changed

3 files changed

+7
-12
lines changed

.github/workflows/build-next.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ jobs:
3636
- name: Check for builder-related changes
3737
id: check
3838
run: |
39-
if git diff --name-only HEAD^ HEAD | grep -E '^(package.json|pnpm-lock.yaml|build/Containerfile.builder|.github/workflows/next-build.yaml)$'; then
39+
if git diff --name-only HEAD^ HEAD | grep -E '^(package.json|pnpm-lock.yaml|build/Containerfile.builder|.github/workflows/build-next.yaml)$'; then
4040
echo "builder_required=true" >> $GITHUB_OUTPUT
4141
else
4242
echo "builder_required=false" >> $GITHUB_OUTPUT

build/Containerfile

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,14 +40,11 @@ RUN pnpm install && \
4040
RUN mkdir /opt/app-root/extension && \
4141
cp -r dist /opt/app-root/extension/ && \
4242
cp package.json /opt/app-root/extension/ && \
43-
cp ./*.png /opt/app-root/extension/
44-
45-
COPY package.json /opt/app-root/extension/
46-
COPY LICENSE /opt/app-root/extension/
47-
COPY README.md /opt/app-root/extension/
48-
COPY icon.png /opt/app-root/extension/
49-
COPY packages/backend/bootable.woff2 /opt/app-root/extension/
50-
COPY packages/backend/media/ /opt/app-root/extension/media
43+
cp ./*.png /opt/app-root/extension/ && \
44+
cp LICENSE /opt/app-root/extension/ && \
45+
cp README.md /opt/app-root/extension/ && \
46+
cp packages/backend/bootable.woff2 /opt/app-root/extension/ && \
47+
cp packages/backend/media/ /opt/app-root/extension/media
5148

5249
# TEMPORARY. Permanent fix will be in the future when we can add all of this to vite script.
5350
# We require the macadam.js binaries and library, so we will manually copy this over to the container image.

build/Containerfile.builder

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,7 @@ ENV HOME=/opt/app-root
2222

2323
# copy the application files to the /opt/app-root/extension-source directory
2424
WORKDIR /opt/app-root/extension-source
25-
RUN mkdir -p /opt/app-root/extension-source
26-
COPY package.json /opt/app-root/extension-source/
27-
COPY pnpm-lock.yaml /opt/app-root/extension-source/
25+
COPY package.json pnpm-lock.yaml /opt/app-root/extension-source/
2826

2927
RUN npm install --global pnpm@10 && \
3028
pnpm --frozen-lockfile install

0 commit comments

Comments
 (0)