Skip to content

Add Smarty php8.4 support#412

Merged
totten merged 2 commits intocivicrm:masterfrom
eileenmcnaughton:smarty58.4
Aug 8, 2024
Merged

Add Smarty php8.4 support#412
totten merged 2 commits intocivicrm:masterfrom
eileenmcnaughton:smarty58.4

Conversation

@eileenmcnaughton
Copy link
Copy Markdown
Contributor

This patch is from a currently open PR
smarty-php/smarty#1043

I think we can merge this to see our 8.4 start running & be relaxed if we lose it temporarily
if we update Smarty again now before it is merged (which seems unlikely but we can hopefully hustle the merge)

Note patch includes an update to latest released smarty which I can rebase out once merged #411

This patch is from a currently open PR
smarty-php/smarty#1043

I think we can merge this to see our 8.4 start running & be relaxed if we lose it temporarily
if we update Smarty again now before it is merged (which seems unlikely but we can hopefully hustle the merge)
@civibot
Copy link
Copy Markdown

civibot bot commented Aug 6, 2024

(Standard links)

@civibot civibot bot added the master label Aug 6, 2024
@totten totten merged commit f6d69c2 into civicrm:master Aug 8, 2024
@totten totten deleted the smarty58.4 branch August 9, 2024 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants