-
-
Notifications
You must be signed in to change notification settings - Fork 427
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 to report results to Gitea review comments #1617
Add support to report results to Gitea review comments #1617
Conversation
4ab655f
to
08c9f9f
Compare
08c9f9f
to
837c163
Compare
6e97ce7
to
ace01eb
Compare
ace01eb
to
3757936
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.
LGTM except the comment on Readme.
@lafriks I don't have any access to gitia. Did you test the code actually works with gitia? |
I'm also a maintainer of Gitea, so I did a testing on my computer with the dev version. Are there any examples with integration tests? Is it possible to start a docker based service in GitHub actions? |
If you want to test you can also do that on https://try.gitea.io/ but I would not use it for integration tests |
362535e
to
7d36daf
Compare
For GitLab, we have an integration test. We created a synced repo and use GitLab CI. https://github.com/reviewdog/reviewdog/blob/master/.gitlab-ci.yml |
Thank you for your work!👍 |
I will see if I can do about implementing tests, at minimum unit tests |
Fixes #1295
Should probably wait for gitea/go-sdk#638