-
Notifications
You must be signed in to change notification settings - Fork 27
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
Execute units tests against php 8.1 & 8.2 #888
Conversation
80a1672
to
eec27aa
Compare
Executes unit tests against more recent php versions on master and stable27 Resolves : #868 Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
eec27aa
to
7dd67de
Compare
@@ -39,7 +39,7 @@ jobs: | |||
strategy: | |||
matrix: | |||
php-versions: ['8.0', '8.1', '8.2'] | |||
server-versions: ['master'] | |||
server-versions: ['master', 'stable27'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why stable27
this is only for the master branch, stable27
should only be tested on its own branch.
(so backporting this to stable27
instead)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that is about adding 8.2 also to the stable27
branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, what susnux says
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically #877 fixed master,
but the change should be either backported to stable27, or you manually add 8.2 to all phpunit-* files tehre
@@ -38,8 +38,8 @@ jobs: | |||
|
|||
strategy: | |||
matrix: | |||
php-versions: ['8.0'] | |||
server-versions: ['master'] | |||
php-versions: ['8.0', '8.1', '8.2'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should also not be changed, to keep the CI matrix small.
Closing this with : #916 |
Executes unit tests against more recent php versions on master and stable27
Resolves : #868