Skip to content

Commit

Permalink
chore: add superlint fixes
Browse files Browse the repository at this point in the history
Added fixes for issues pointed out by the 'superlint' workflow
  • Loading branch information
vindelicus committed Nov 14, 2024
1 parent 7450842 commit 7dbbcd2
Show file tree
Hide file tree
Showing 5 changed files with 779 additions and 100 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/superlint.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: 'Run Super-Linter'
name: "Run Super-Linter"

on: ['push', 'pull_request']
on: ["push", "pull_request"]

permissions: read-all

Expand All @@ -26,3 +26,4 @@ jobs:
DEFAULT_BRANCH: main
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
VALIDATE_JSCPD: false
BASH_EXEC_IGNORE_LIBRARIES: true
7 changes: 6 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ ARG PORT=9001
ARG USER=lhci

# persistent / runtime deps
# hadolint ignore=DL3008
# hadolint ignore=DL3018
RUN apk update && apk add --no-cache --update \
python3=~3.12 \
build-base=~0.5 \
Expand All @@ -42,6 +42,11 @@ COPY --chmod=755 docker/bin/lhctl /usr/local/bin
COPY --chmod=755 docker/bin/lhci /usr/local/bin
COPY --chmod=644 docker/lib/utils.sh /usr/local/lib

# add a healthcheck
# ref: https://developer.shopware.com/docs/guides/hosting/installation-updates/cluster-setup.html#health-check
HEALTHCHECK --interval=30s --timeout=10s --start-period=10s --retries=5 \
CMD curl --fail "http://localhost:${PORT:-9001}/healthz" || exit 1

# -------------------------------------
# LHCI Builder Image
# -------------------------------------
Expand Down
Loading

0 comments on commit 7dbbcd2

Please sign in to comment.