[tests-only] Removed unused make target, bump ocis commit id #3627
Conversation
grgprarup
force-pushed
the
remove-unused-make-target
branch
from
d5c50f4 to
b13ef65
Compare
grgprarup
force-pushed
the
remove-unused-make-target
branch
2 times, most recently
from
3bbf894 to
d016c06
Compare
|
Added test by PR owncloud/ocis#5415 are failing for |
That is "correct". The source-code fix would only be in So please add to expected-failures. In this case, there will have to be a heading with no link to any issue. Just give a general description in the heading and list the failing scenarios. I have no idea if this sort of thing will be back-ported to |
grgprarup
force-pushed
the
remove-unused-make-target
branch
from
438f6e0 to
14a2d5b
Compare
grgprarup
requested review from
labkode,
gmgigi96,
glpatcern and
a team
as code owners
|
@labkode @gmgigi96 @glpatcern this makes some further changes to Please review. We can't merge |
|
Looping in @vascoguita as he's fixing the CI and the build infra. The Changelog check should be skipped a priori. |
|
@grgprarup could you please rebase? it seems some conflicts got in the way meanwhile. |
|
PR #3629 changed the CI also. @grgprarup please rebase and sort out what changes actually are still needed here. |
grgprarup
dismissed stale reviews from glpatcern, vascoguita, and phil-davis
via
0d5f2d9
grgprarup
force-pushed
the
remove-unused-make-target
branch
from
14a2d5b to
0d5f2d9
Compare
grgprarup
requested review from
glpatcern and
phil-davis
and removed request for
glpatcern
|
@labkode @gmgigi96 @glpatcern there used to be code in the drone changelog check that would skip the check when we put |
|
@labkode @gmgigi96 @glpatcern ready for review again. |
This PR removes the unused make target from Makefile, same as (#3622),
added APITESTS_REPO_GIT_URL env variable to .drone.env (#3593),
bump ocis commit id to latest
Fix owncloud/ocis#5450
Part of owncloud/QA#795