-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add support for Drone CI #4382
Add support for Drone CI #4382
Conversation
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.
Nice! Thanks for the contribution. Do you know of a convenient way for us, maintainers, to test this? No worries if not, though.
@browniebroke You're welcome! At this moment, unfortunately, no. But still trying to understand a way to somehow figure it out. |
Are there any update on this? If not, I'm happy to accept it as community contributed as is. It's very much self-contained and easy to remove if we change our mind. |
@browniebroke Right now I don't have any updates, but at that moment I checked for some static analyzing tools to ensure that the outputted syntax is correct but didn't find much. If I find anything I'll definitely inform you and support any changes. If it's possible for you to merge, I'll be happy. And actually, this if my first open-source contribution. :) |
Ok, let's get it out, we might get some feedback from people using it, and it shouldn't impact those who don't. |
Description
Added Drone CI support and configuration.
Checklist:
Rationale
Fix #2082