-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
ci: Migrate Litmus and CalDAVTester from drone to GitHub #45619
Conversation
5d835a1
to
5fb1e7b
Compare
/backport to stable29 |
73f26c4
to
a150c29
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, looks good to me!
ERROR: There were 1 missing files:
.drone.yml
ERROR: Please remove or add those files again or inform the release team about those now files to be included or excluded from the release tar ball.
Perhaps removing the file from https://github.com/nextcloud/server/blob/master/build/files-checker.php is sufficient?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you 👏
a150c29
to
4f9dba7
Compare
@@ -8,7 +8,6 @@ | |||
'.', | |||
'..', | |||
'.devcontainer', | |||
'.drone.yml', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
We can finally shut of drone workers. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
4f9dba7
to
dc9df43
Compare
@nickvergessen I resolved you comments, when merging you need to remove drone from the required checks. |
Summary
Migrate the last two remaining CI tests from drone to GitHub CI. Allowing us to shut down drone workers.
TODO
Checklist