Skip to content

Conversation

@thirsch
Copy link
Collaborator

@thirsch thirsch commented Dec 13, 2022

Using the current release, we have found the following issues under php 8.1

@thirsch
Copy link
Collaborator Author

thirsch commented Dec 13, 2022

The testsuite is currently not working under Apple M1. I'll get myself a amd64 test machine and come back with a fix.

@thirsch
Copy link
Collaborator Author

thirsch commented Dec 13, 2022

The testsuite is currently not working under Apple M1. I'll get myself a amd64 test machine and come back with a fix.

Tests are running fine on my local machine and over in the fork of the repository. Any idea, why it crashed here? Can you re-run them manually or do I have to push a change to the branch to start them again?

@thirsch thirsch force-pushed the feature/more-php-8.1-fixes branch from 31eb7a7 to d126433 Compare January 9, 2023 19:16
Copy link
Member

@thePanz thePanz left a comment

Choose a reason for hiding this comment

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

@thirsch : re-run the failing job, it's green now
Could you please rebase this PR?

@thirsch thirsch force-pushed the feature/more-php-8.1-fixes branch from d126433 to 8b81a56 Compare January 9, 2023 21:09
@thirsch
Copy link
Collaborator Author

thirsch commented Jan 9, 2023

@thirsch : re-run the failing job, it's green now Could you please rebase this PR?

done

@thePanz thePanz merged commit 9bba8c5 into FriendsOfSymfony1:master Jan 10, 2023
@thirsch thirsch deleted the feature/more-php-8.1-fixes branch January 12, 2023 18:10
@mkopinsky
Copy link
Contributor

@thirsch were you able to get tests to run on Mac M1?

@thirsch
Copy link
Collaborator Author

thirsch commented Dec 18, 2023

@thirsch were you able to get tests to run on Mac M1?

Sorry for my late reply. To be honest, I don't remember. ;-)

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.

3 participants