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 commit id for tests #3639

Merged

Conversation

SwikritiT
Copy link
Contributor

Part of: owncloud/QA#797

Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Note: the test scenarios are being cleaned-up, irrelevant test scenarios removed from the oCIS test code. So that is why quite a lot of items are removed from expected-failures. They are test scenarios that are not relevant to ocis/reva anyway.

@phil-davis phil-davis marked this pull request as ready for review February 3, 2023 10:24
@phil-davis phil-davis requested review from labkode, gmgigi96, glpatcern and a team as code owners February 3, 2023 10:24
@phil-davis phil-davis merged commit 9f75d9e into cs3org:master Feb 3, 2023
@phil-davis phil-davis deleted the bump-commit-id-for-tests-20230203-master branch February 3, 2023 10:26
vascoguita pushed a commit to Tomaszal/reva that referenced this pull request Feb 6, 2023
* Bump commit id for tests

* update expected to fail file
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