-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
Create a first draft for .drone.yml #96
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
Conversation
|
The image is my image and it's based on go shipped with alpine edge, should be fine for now. Cgo should be disabled for now. For the lint we got an open issue :) |
tboerger
left a comment
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.
I will add the required secrets later on.
.drone.yml
Outdated
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.
gitea/gitea
.drone.yml
Outdated
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.
gitea/gitea
.drone.yml
Outdated
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.
coverage.gitea.io
.drone.yml
Outdated
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.
0
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.
🎉
.drone.yml
Outdated
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.
Fine for now, we will add a matrix for that later
|
drone currently blocks because: |
.drone.yml
Outdated
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.
Just add a line like apk -U add openssh-client to fix the blocking about missing ssh-keygen.
.drone.yml
Outdated
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.
@lunny should we setup a simple instance of Aircover for coverage reports? It's from the same author as LGTM and a simple docker container.
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.
why use Aircover not codecov-io?
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.
Because aircover is another nearly oss coverage tool we can also provide.
Just saying that because there is already a native plugin for that available.
We can also add gitea integration to that as well.
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.
And it's not adding these annoying comments :P
Current coverage is 2.24% (diff: 100%)
|
|
Drone passes now and is freakin' fast compared to travis 🎉 |
|
LGTM, so you don't miss one when needed :) |
|
You should also install Beside that you can also integrate a matrix build for |
|
LGTM |
|
@metalmatze any reason for not using the official |
|
The official image doesn't have make on board, |
* Fixes go-gitea#87 - Make Hashtags Hyperlinks in HTML view * Added hashtag_test.go for testing
Signed-off-by: Duncan Ogilvie <[email protected]>
About a few things I'm not sure.
webhippie/golang:edgeas build imageCGO_ENABLED: 1?make lintbecause it throws > 500 errors.