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(cypress): Also clone 3rdparty submodule to keep in sync for cypress tests #41135

Merged
merged 1 commit into from Oct 27, 2023

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Oct 26, 2023

Summary

If lib uses updated 3rdparty libraries but the shallow server is outdated (like now the server is 28 days old -> dependencies have been changed) it might lead to error 500 as the Nextcloud server can not find that changed dependencies.

So we need to also pull submodules on Cypress init.

Checklist

…ss tests

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@solracsf solracsf added this to the Nextcloud 28 milestone Oct 27, 2023
@nickvergessen
Copy link
Member

Failing node test seems unrelated

@nickvergessen nickvergessen merged commit 5f269f0 into master Oct 27, 2023
44 of 46 checks passed
@nickvergessen nickvergessen deleted the fix/cypress-ci branch October 27, 2023 07:19
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 bug CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants