Skip to content
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

Add example of how to configure php for el9 based systems #811

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

mvangoor
Copy link
Contributor

Description

Add test kitchen coverage for el9 based systems using php fpm

Issues Resolved

Missing php coverage for el9 based systems

Check List

  • A summary of changes made is included in the CHANGELOG under ## Unreleased
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@mvangoor mvangoor requested a review from a team as a code owner July 30, 2024 08:29
@xorima xorima added the Release: Patch Release to Chef Supermarket as a version patch when merged label Jul 30, 2024
bmhughes
bmhughes previously approved these changes Jul 30, 2024
Copy link

@bmhughes bmhughes left a comment

Choose a reason for hiding this comment

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

LGTM!

@mvangoor Can you fixup the markdown failure and we can get this merged. Thanks!

Drop centos stream 8, ubuntu 18.04 and Debian 10

Signed-off-by: Mike van Goor <[email protected]>
@mvangoor mvangoor force-pushed the main branch 6 times, most recently from c0ec46d to f457199 Compare July 31, 2024 13:29
@ramereth
Copy link
Contributor

ramereth commented Aug 1, 2024

@mvangoor can you please fix the fedora test that's failing or maybe exclude it at least?

@ramereth ramereth merged commit 5c2dd91 into sous-chefs:main Aug 2, 2024
134 checks passed
@kitchen-porter
Copy link
Contributor

Released as: 9.3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release: Patch Release to Chef Supermarket as a version patch when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants