Skip to content

Fix master checks#2991

Draft
christianlupus wants to merge 3 commits intomasterfrom
maintenance/fix-master-checks
Draft

Fix master checks#2991
christianlupus wants to merge 3 commits intomasterfrom
maintenance/fix-master-checks

Conversation

@christianlupus
Copy link
Collaborator

@christianlupus christianlupus commented Feb 4, 2026

Topic and Scope

This should make sure the unit test are again running as intended also on the master branch of NC server.

Concerns/issues

This is a regression fixing that should have been part of the tests in any case.

Formal requirements

There are some formal requirements that should be satisfied. Please mark those by checking the corresponding box.

  • I did check that the app can still be opened and does not throw any browser logs
  • I created tests for newly added PHP code (check this if no PHP changes were made)
  • I updated the OpenAPI specs and added an entry to the API changelog (check if API was not modified)
  • I notified the matrix channel if I introduced an API change

Signed-off-by: Christian Wolf <github@christianwolf.email>
Signed-off-by: Christian Wolf <github@christianwolf.email>
Signed-off-by: Christian Wolf <github@christianwolf.email>
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

Test Results

   12 files    592 suites   2m 19s ⏱️
  589 tests   584 ✅ 1 💤 4 ❌
2 356 runs  2 347 ✅ 5 💤 4 ❌

For more details on these failures, see this check.

Results for commit 4cce623.

♻️ This comment has been updated with latest results.

@christianlupus christianlupus force-pushed the maintenance/fix-master-checks branch 2 times, most recently from 7176e45 to 4cce623 Compare February 8, 2026 18:44
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.

1 participant