-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch docker image to new 8.x-php8.4 tag. #114
Conversation
@@ -41,23 +41,10 @@ jobs: | |||
- name: Code sniff | |||
run: composer run-script code:lint | |||
|
|||
- name: Install Terminus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is terminus already installed at another step?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's in the docker image
|
||
- name: Install Terminus Build Tools | ||
run: | | ||
t3 self:plugin:install terminus-build-tools-plugin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are used so in some of the Robo workflows. Is there some other place that the build tools are being installed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's in the docker image
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If these things are already in the image then this looks g2g to me. 👍
I think we should be using a Pantheon official image and ideally, that image should contain terminus and build tools in it. I'm switching the image in this PR.