Skip to content

fix(lint): apply new nextcloud code style#8011

Merged
grnd-alt merged 2 commits into
mainfrom
fix/update-code-style
Jun 2, 2026
Merged

fix(lint): apply new nextcloud code style#8011
grnd-alt merged 2 commits into
mainfrom
fix/update-code-style

Conversation

@grnd-alt
Copy link
Copy Markdown
Member

@grnd-alt grnd-alt commented Jun 2, 2026

dependabot auto-merged a phpunit bump to 13.x which is not compatible with php8.2 so that has to be reverted.
the other changes outside tests/integration all come from composer cs:fix as nextcloud codestyle changed.

Signed-off-by: grnd-alt <git@belakkaf.net>
@grnd-alt grnd-alt requested a review from luka-nextcloud as a code owner June 2, 2026 07:20
This reverts commit 7d94044.

Signed-off-by: grnd-alt <git@belakkaf.net>
@grnd-alt grnd-alt force-pushed the fix/update-code-style branch from 74fcc8c to 1b602e6 Compare June 2, 2026 07:26
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 2, 2026

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 97460 was 93102 (+4.68%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 2, 2026

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 97480 was 93102 (+4.7%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@silverkszlo silverkszlo self-requested a review June 2, 2026 07:41
@grnd-alt grnd-alt merged commit 7ad6945 into main Jun 2, 2026
43 checks passed
@grnd-alt grnd-alt deleted the fix/update-code-style branch June 2, 2026 07:49
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