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 dynamic properties and other problems in tests for PHP 8.2 #35156

Merged
merged 5 commits into from
Nov 15, 2022

Conversation

come-nc
Copy link
Contributor

@come-nc come-nc commented Nov 14, 2022

See #32595
See #32463

Signed-off-by: Côme Chilliet [email protected]

@come-nc come-nc self-assigned this Nov 14, 2022
Signed-off-by: Côme Chilliet <[email protected]>
@come-nc come-nc marked this pull request as ready for review November 14, 2022 16:09
@come-nc come-nc requested review from a team, ArtificialOwl, blizzz and skjnldsv and removed request for a team November 14, 2022 16:09
@come-nc come-nc added the 3. to review Waiting for reviews label Nov 14, 2022
@come-nc come-nc added this to the Nextcloud 26 milestone Nov 14, 2022
@come-nc come-nc enabled auto-merge November 14, 2022 16:22
Copy link
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

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

a few remarks, also CI isn't convinced.

Many editors remove trailing whitespaces, so avoid depending on them for
 tests success.

Signed-off-by: Côme Chilliet <[email protected]>
@come-nc come-nc requested review from tcitworld and szaimen November 15, 2022 11:15
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@come-nc come-nc merged commit 4893d5e into master Nov 15, 2022
@come-nc come-nc deleted the fix/fix-tests-for-8.2 branch November 15, 2022 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants