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.

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

fix: use php 8.1 as that is required now #3610

Closed
wants to merge 3 commits into from

Conversation

max-nextcloud
Copy link
Collaborator

📝 Summary

Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud
Copy link
Collaborator Author

I think current master is segfaulting with php 7.4, 8.0 and 8.1.

I'll try to spin up a current master in my local dev env.

@max-nextcloud
Copy link
Collaborator Author

I'll try to spin up a current master in my local dev env.

That worked fine 🤔

@max-nextcloud
Copy link
Collaborator Author

  • c31e0e3f45ce5fd70d4f74b8cb7a0a0a445ad5a3 was the server commit of the last CI run that succeeded.
  • b90ccaac0f7abe85be3f4efa974d96296c3a250f was the first that failed.

@cypress
Copy link

cypress bot commented Dec 28, 2022



Test summary

116 0 0 0Flakiness 0


Run details

Project Text
Status Passed
Commit af9d301
Started Dec 29, 2022 1:02 PM
Ended Dec 29, 2022 1:08 PM
Duration 05:24 💡
OS Linux Ubuntu -
Browser Electron 106

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

Signed-off-by: Max <max@nextcloud.com>
lets see if the segfault happens in the
generateAvatarFromSvg function.

Signed-off-by: Max <max@nextcloud.com>
@juliushaertl
Copy link
Member

Closing as the server PR was reverted

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.

2 participants