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

[full-ci][tests-only]Bump ocis commit id edge #3872

Merged
merged 3 commits into from
May 11, 2023

Conversation

PrajwolAmatya
Copy link
Contributor

This PR bumps latest commit id from ocis
Part of owncloud/QA#807

@PrajwolAmatya PrajwolAmatya force-pushed the bump-ocis-commit-id-edge branch from 76cdc73 to a4c7599 Compare May 10, 2023 06:41
@phil-davis phil-davis self-requested a review May 10, 2023 07:24
@phil-davis phil-davis changed the title [full-ci][test-only]Bump ocis commit id edge [full-ci][tests-only]Bump ocis commit id edge May 10, 2023
@cs3org cs3org deleted a comment from update-docs bot May 10, 2023
@PrajwolAmatya PrajwolAmatya force-pushed the bump-ocis-commit-id-edge branch from a4c7599 to ee51d44 Compare May 10, 2023 08:33
@phil-davis
Copy link
Contributor

@PrajwolAmatya somehow GitHub is reporting conflicts - so that needs to be sorted out.

@grgprarup
Copy link
Contributor

@PrajwolAmatya somehow GitHub is reporting conflicts - so that needs to be sorted out.

In this PR #3849, drone env api test url, branch and commit id is changed to aduffeck's ocis repo. Is that intentional change or forgot to revert back to owncloud repo.

@saw-jan
Copy link
Contributor

saw-jan commented May 10, 2023

In this PR #3849, drone env api test url, branch and commit id is changed to aduffeck's ocis repo. Is that intentional change or forgot to revert back to owncloud repo.

I think, we can revert to ocis repo after reva bump in ocis.

@phil-davis
Copy link
Contributor

I will rebase and put in the latest commit id etc.
Give me a few minutes.

@phil-davis phil-davis force-pushed the bump-ocis-commit-id-edge branch from ee51d44 to 886f4ad Compare May 11, 2023 09:49
@phil-davis
Copy link
Contributor

Rebase done - let's see what CI says.

@phil-davis
Copy link
Contributor

I will push again in a few minutes

@phil-davis phil-davis force-pushed the bump-ocis-commit-id-edge branch from 886f4ad to 3a174a8 Compare May 11, 2023 10:31
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 if CI passes

@phil-davis phil-davis marked this pull request as ready for review May 11, 2023 10:38
@phil-davis phil-davis requested review from labkode, ishank011, glpatcern and a team as code owners May 11, 2023 10:38
@phil-davis phil-davis merged commit 562ede3 into cs3org:edge May 11, 2023
@phil-davis phil-davis deleted the bump-ocis-commit-id-edge branch May 11, 2023 11:23
2403905 pushed a commit to 2403905/reva that referenced this pull request May 19, 2023
* bump latest commit id

* standardize indent of expected-failures files

* Add ocis issue 3544 to expected-failures

---------

Co-authored-by: Phil Davis <[email protected]>
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.

4 participants