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

[tests-only] [full-ci] bump latest ocis commit id to edge #4234

Merged

Conversation

KarunAtreya
Copy link
Contributor

Part of owncloud/QA#832

@phil-davis phil-davis merged commit 022a0b5 into cs3org:edge Oct 6, 2023
7 checks passed
@phil-davis phil-davis deleted the bump-latest-ocis-commitid-to-edge branch October 6, 2023 06:58
@KarunAtreya KarunAtreya restored the bump-latest-ocis-commitid-to-edge branch October 9, 2023 09:50
@KarunAtreya KarunAtreya deleted the bump-latest-ocis-commitid-to-edge branch October 9, 2023 09:51
@rhafer
Copy link
Contributor

rhafer commented Oct 10, 2023

@KarunAtreya @phil-davis For some reason the drone ci has not been run on this one. (This should be fixed meanwhile)

Unfortunately this breaks the acceptance tests. See e.g.: https://drone.owncloud.com/cs3org/reva/3148/15/6
Can we please revert this for now?

cc @aduffeck

@saw-jan
Copy link
Contributor

saw-jan commented Oct 10, 2023

I have an idea about the failure. Will do a fix PR

@phil-davis
Copy link
Contributor

For some reason the drone ci has not been run on this one.

I guess that I saw the PR result "all checks have passed".

Maybe we should make some of the checks "required" so that GitHub won't let us merge without those results existing and passing.

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.

None yet

4 participants