Skip to content

Fix Workflow use actions/cache@v4 instead of V1 or V2 - #97

Merged
Touxten merged 4 commits into
PrestaShop:devfrom
Touxten:Touxten-patch-1
Oct 9, 2025
Merged

Fix Workflow use actions/cache@v4 instead of V1 or V2#97
Touxten merged 4 commits into
PrestaShop:devfrom
Touxten:Touxten-patch-1

Conversation

@Touxten

@Touxten Touxten commented Jul 21, 2025

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Fix Workflow
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket?
How to test? CI green
Sponsor company @Touxten

@ps-jarvis

Copy link
Copy Markdown

Hello @Touxten!

This is your first pull request on ps_shoppingcart repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Jul 21, 2025
@Touxten
Touxten marked this pull request as draft July 21, 2025 07:56
@Hlavtox

Hlavtox commented Jul 21, 2025

Copy link
Copy Markdown
Member

@Touxten

* @return array presented cart

return array|CartLazyArray

@Touxten
Touxten marked this pull request as ready for review July 21, 2025 08:33
@Touxten

Touxten commented Jul 21, 2025

Copy link
Copy Markdown
Contributor Author

Thx you @Hlavtox it's ok

@Touxten
Touxten marked this pull request as draft July 21, 2025 09:46
@Hlavtox

Hlavtox commented Jul 21, 2025

Copy link
Copy Markdown
Member

Aaaaargh, just delete the typehit.

If we add use PrestaShop\PrestaShop\Adapter\Presenter\Cart\CartLazyArray;, it won't be available in older versions of prestshop and it will complain again or may not work.

@Touxten
Touxten marked this pull request as ready for review July 21, 2025 13:07
@SiraDIOP

SiraDIOP commented Oct 9, 2025

Copy link
Copy Markdown

Hello @Touxten,

Thanks for your PR, CI are green ^^

@SiraDIOP SiraDIOP self-assigned this Oct 9, 2025
@SiraDIOP SiraDIOP added the QA ✔️ Status: Check done, Code approved label Oct 9, 2025
@Touxten
Touxten merged commit 7de52f4 into PrestaShop:dev Oct 9, 2025
9 checks passed
@Touxten

Touxten commented Oct 9, 2025

Copy link
Copy Markdown
Contributor Author

Thank you @SiraDIOP

@github-project-automation github-project-automation Bot moved this from Ready for review to Merged in PR Dashboard Oct 9, 2025
@ps-jarvis

Copy link
Copy Markdown

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA ✔️ Status: Check done, Code approved

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants