Skip to content

Conversation

@christianlupus
Copy link
Collaborator

Topic and Scope

This is a quick update of the test infrastructure to run any tests against the current stable 33 branch of NC server.

Concerns/issues

I had to mark the app as compatible with NC33 to make the test run. Thus, this should not be merged until support with NC 33 is actually established.

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>
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Test Results

   12 files    592 suites   2m 29s ⏱️
  589 tests   588 ✅ 1 💤 0 ❌
2 356 runs  2 351 ✅ 5 💤 0 ❌

Results for commit 39c03ab.

♻️ This comment has been updated with latest results.

@christianlupus christianlupus force-pushed the maint/test-against-nc-33 branch from 97ec514 to 39c03ab Compare February 2, 2026 18:30
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