Skip to content

Conversation

erikwebb
Copy link
Member

@erikwebb erikwebb commented Sep 25, 2025

For customers that do not (or prefer not) to have a local PHP environment like potentially Next.js developers, an alternative installation method via Docker avoids any need for local setup.

  • Install all necessary dependencies into Docker build and run layers
  • Using PHP 8.4 for Docker image
  • Add README instructions to utilize standalone Docker container method with notes on using Docker volume for config persistence

Specifically, the local flow is -

docker build . -t terminus
docker volume create terminus --ignore
docker run -tv terminus:/root/.terminus terminus:latest self:info

Presumably, the user flow would be -

docker volume create terminus --ignore
alias terminus="docker run -tv terminus:/root/.terminus pantheon/terminus:4"
terminus self:info

@wiz-inc-b08cf2810f
Copy link

wiz-inc-b08cf2810f bot commented Sep 25, 2025

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data -
IaC Misconfiguration IaC Misconfigurations 2 Medium 3 Low
Total 2 Medium 3 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@erikwebb erikwebb marked this pull request as ready for review September 25, 2025 21:25
@erikwebb erikwebb requested a review from a team as a code owner September 25, 2025 21:26
@erikwebb erikwebb requested a review from CobyPear September 25, 2025 21:39
Copy link
Member

@CobyPear CobyPear left a comment

Choose a reason for hiding this comment

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

non-blocking comment. The Dockerfile part looks good to me.

Comment on lines +52 to +62
build_docker_image:
runs-on: ubuntu-latest
name: Build Docker image
steps:
- name: Checkout
uses: actions/checkout@v3
- name: Build Docker image
run: docker build . -t terminus
- name: Run built Docker image
run: docker run terminus art

Copy link
Member

Choose a reason for hiding this comment

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

Might be worth holding of on this until we have a public registry we can push to or until this repo has been onboarded to the WIF stuff so we can use the shared actions.

@erikwebb erikwebb force-pushed the erikwebb/terminus-Dockerfile branch from e22ed0d to ce4b1d4 Compare September 26, 2025 20:14
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.

2 participants