Skip to content
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

Merged
merged 4 commits into from
Dec 6, 2023

Conversation

lafriks
Copy link
Contributor

@lafriks lafriks commented Dec 2, 2023

  • Updated Unreleased section in CHANGELOG or it's not notable changes.

Fixes #1295

attels

Should probably wait for gitea/go-sdk#638

service/gitea/gitea.go Outdated Show resolved Hide resolved
@lafriks lafriks force-pushed the feat/gitea_pr_review branch 2 times, most recently from 4ab655f to 08c9f9f Compare December 2, 2023 21:34
@lafriks lafriks force-pushed the feat/gitea_pr_review branch from 08c9f9f to 837c163 Compare December 2, 2023 21:46
@lafriks lafriks requested a review from haya14busa December 3, 2023 15:31
@lafriks lafriks force-pushed the feat/gitea_pr_review branch from 6e97ce7 to ace01eb Compare December 4, 2023 15:07
@lafriks lafriks force-pushed the feat/gitea_pr_review branch from ace01eb to 3757936 Compare December 4, 2023 15:19
Copy link
Member

@haya14busa haya14busa left a 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.

README.md Outdated Show resolved Hide resolved
@haya14busa
Copy link
Member

@lafriks I don't have any access to gitia. Did you test the code actually works with gitia?
Can we potentially have some integration tests?

@lafriks
Copy link
Contributor Author

lafriks commented Dec 5, 2023

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?

@lafriks
Copy link
Contributor Author

lafriks commented Dec 5, 2023

If you want to test you can also do that on https://try.gitea.io/ but I would not use it for integration tests

@lafriks lafriks requested a review from haya14busa December 5, 2023 11:34
@lafriks lafriks force-pushed the feat/gitea_pr_review branch from 362535e to 7d36daf Compare December 5, 2023 14:36
@haya14busa
Copy link
Member

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

@haya14busa haya14busa merged commit 962b3f0 into reviewdog:master Dec 6, 2023
14 of 15 checks passed
@lafriks lafriks deleted the feat/gitea_pr_review branch December 6, 2023 15:29
@haya14busa
Copy link
Member

Thank you for your work!👍

@lafriks
Copy link
Contributor Author

lafriks commented Dec 6, 2023

I will see if I can do about implementing tests, at minimum unit tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Support of Gitea as Reporter
2 participants