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.

Sign up for GitHub

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

Jump to bottom

[CI] IBX-4515: Include tests running on PHP 8.2 #129

Merged
merged 1 commit into from
Dec 21, 2023
Merged

[CI] IBX-4515: Include tests running on PHP 8.2 #129

merged 1 commit into from
Dec 21, 2023

Conversation

micszo
Copy link
Contributor

@micszo micszo commented Dec 12, 2023 •
edited
Loading

Question Answer
JIRA issue IBX-4515
Type CI
Target Ibexa version v3.3
BC breaks no
Doc needed ???

Replaced PHP 8.1 with 8.2.

Which gives for v3.3:
7.3, 7.4, 8.2
for v4:
7.4, 8.0, 8.2.

Other editions:
ibexa/content#118
ibexa/experience#296
ibexa/commerce#529

Test PRs for higher versions:
#131
#132
ibexa/content#121
ibexa/headless#42
ibexa/commerce#543
ibexa/commerce#544

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Checked that target branch is set correctly.
  • Asked for a review (ping @ibexa/engineering).

@micszo micszo marked this pull request as ready for review December 21, 2023 10:35
mnocon
mnocon approved these changes Dec 21, 2023
View reviewed changes
Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

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

Thanks!

@micszo micszo merged commit 5171945 into 3.3 Dec 21, 2023
9 checks passed
@micszo micszo deleted the ibx-4515 branch December 21, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mnocon mnocon mnocon approved these changes

@tomaszszopinski tomaszszopinski tomaszszopinski approved these changes

@bogusez bogusez bogusez approved these changes

@barbaragr barbaragr Awaiting requested review from barbaragr

@katarzynazawada katarzynazawada Awaiting requested review from katarzynazawada

Assignees
No one assigned
Labels
Ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@micszo @mnocon @tomaszszopinski @bogusez