Skip to content

FIX Explicitly add php ppa - #175

Merged
blueo merged 1 commit into
silverstripe:2.0from
creative-commoners:pulls/2.0/fix-ci
Jun 23, 2026
Merged

FIX Explicitly add php ppa#175
blueo merged 1 commit into
silverstripe:2.0from
creative-commoners:pulls/2.0/fix-ci

Conversation

@emteknetnz

@emteknetnz emteknetnz commented Jun 23, 2026

Copy link
Copy Markdown
Member

@emteknetnz
emteknetnz marked this pull request as ready for review June 23, 2026 04:11

@blueo blueo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

change lgtm. Minor wording suggestion

Comment thread .github/workflows/ci.yml Outdated
Comment on lines +271 to +273
# ppa:ondrej/apache2 is not added by default.
# ppa:ondrej/php must also be added explicitly - while shivammathur/setup-php installs the PHP CLI,
# it does not leaves the PPA registered in apt sources

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
# ppa:ondrej/apache2 is not added by default.
# ppa:ondrej/php must also be added explicitly - while shivammathur/setup-php installs the PHP CLI,
# it does not leaves the PPA registered in apt sources
# Neither ppa:ondrej/php nor ppa:ondrej/apache2 are added by default.
# ppa:ondrej/php must be added explicitly - while shivammathur/setup-php installs the PHP CLI,
# it does not leave the PPA registered in apt sources, which is needed for libapache2-mod-php below.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated

@blueo
blueo merged commit f3431eb into silverstripe:2.0 Jun 23, 2026
2 checks passed
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.

3 participants