Skip to content

Remove PHP 7 from template store - #168

Merged
alexellis merged 1 commit into
masterfrom
remove-php7-template
Oct 21, 2026
Merged

alexellis merged 1 commit into
masterfrom
remove-php7-template

Conversation

@alexellis

Copy link
Copy Markdown
Member

PHP 7 is end of life and is being removed from the maintained OpenFaaS templates in openfaas/templates#325. Remove its Template Store entry while retaining PHP 8 as the official, recommended PHP template.

Validation:

  • Parsed templates.json with jq.
  • Confirmed there are no remaining PHP 7 entries.
  • Confirmed PHP 8 remains official and recommended.

PHP 7 is end of life and no longer belongs in the list of supported
OpenFaaS templates. Remove its entry while retaining PHP 8 as the
recommended PHP template.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@reviewfn

reviewfn Bot commented Oct 21, 2026

Copy link
Copy Markdown

AI Pull Request Overview

Disclaimer: This review was generated by automated AI and may contain errors. Do not trust its outputs without human verification.

Summary

  • Removes the end-of-life php7 template from the Template Store.
  • Retains php8 as an official and recommended PHP template.
  • The scoped commit contains no prohibited AI authorship or attribution indicators.
  • No blocking findings identified.

Approval rating (1-10)

10/10. The focused removal is internally consistent and preserves the supported PHP replacement.

Summary per file

Summary per file
File path Summary
templates.json Removes the PHP 7 entry while retaining recommended PHP 8 support.

Overall Assessment

The change is appropriately scoped to removing the obsolete Template Store entry. The remaining PHP 8 entry continues to be marked official and recommended, and no repository references to PHP 7 remain.

Detailed Review

Detailed Review

No findings.

AI agent details.

Agent processing time: 42.564s
Environment preparation time: 3.332s
Total time from webhook: 49.064s

@alexellis
alexellis merged commit 579f365 into master Oct 21, 2026
3 checks passed
@alexellis
alexellis deleted the remove-php7-template branch October 21, 2026 09:19
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.

1 participant