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

Implement CodeClimate (self-hosted) #5267

Closed
Extarys opened this issue Nov 4, 2018 · 11 comments
Closed

Implement CodeClimate (self-hosted) #5267

Extarys opened this issue Nov 4, 2018 · 11 comments
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/enhancement An improvement of existing functionality

Comments

@Extarys
Copy link

Extarys commented Nov 4, 2018

Description

Since CodeCLimate integration is a paid feature of GitLab, it would be great if Gitea would find a way to implement CodeClimate code quality check on merging and maybe add a Stats tab to the repo to see the code coverage (and in the future maybe vulnerability scan, etc)

It would be a great addition since there is currently no self hosted software that do it for free (GitLab, I'm looking at you!)

I hope this is a good place to make such suggestion.

@lunny
Copy link
Member

lunny commented Nov 4, 2018

Is the API and webhooks enough to integrate with CodeCLimate?

@Extarys
Copy link
Author

Extarys commented Nov 4, 2018

Does gitea feature Code Quality feedback on merge request? I haven't started testing anything yet but couldn't find anyone on the web talking about gitea and CodeClimate and I'm new to CI/CD. I was looking for a GitLab equivalent that would gives me a lot of feedback on my repo's status.

@davidak
Copy link

davidak commented Nov 6, 2018

That would be indeed nice to have, but maybe better as stand-alone project, so it can also support other code hosting plattforms.

There are other sites like CodeClimate, some even open source:

For code quality check AFTER each commit, you can use a CI like drone, but a service like CodeClimate can even check a PR BEFORE it is merged and even prevent merging of broken code. It can also provide nice statistics about the code quality over time!

@lunny
Copy link
Member

lunny commented Nov 7, 2018

I think status API could be used by this service to do that like Drone or jenkins to do. Gitea has implemented status API but need more work on pull request UI.

@Extarys
Copy link
Author

Extarys commented Nov 10, 2018

It would be awesome to be able to have something that can have an all-in-one free open source alternative. Right now most services can be linked to GitHub but it is not self-hosted. Some features of GitLab is paid and quite heavy.

@davidak
Copy link

davidak commented Nov 10, 2018

I just found a CI for Go that is open source (but only seem to work with GitHub). It might can get extended to work with other languages and code hosting plattforms. https://golangci.com/

But maybe to have a sane solution we should focus on integrating ANY CI with ANY code analysis tool for ANY language and just search for or build a plattform to collect and display results, like https://codecov.io/ and https://coveralls.io/ for code coverage. It shouldn't be too complex to replicate the statistic features of CodeClimate and others.

@stale
Copy link

stale bot commented Jan 9, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Jan 9, 2019
@Extarys
Copy link
Author

Extarys commented Jan 10, 2019

@davidak We could take the golangci open source version and include it in Gitea 🗡️ Nice research!

@stale stale bot removed the issue/stale label Jan 10, 2019
@stale
Copy link

stale bot commented Mar 11, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Mar 11, 2019
@lunny lunny added the type/enhancement An improvement of existing functionality label Mar 11, 2019
@stale stale bot removed the issue/stale label Mar 11, 2019
@lunny lunny added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Mar 11, 2019
@alexanderadam
Copy link

alexanderadam commented Oct 23, 2019

There's also Pronto, a self hosted tool for a variety of languages, frameworks and checks that might be relevant in this list.

For a proper support (like the experience users have on other services) there should be PR support through the API (like approvals and rejects — maybe even per line)). Otherwise we can "only" write comments to PRs.
And it obviously also need some integration from pronto itself but I guess wouldn't be too difficult because there are various possibilities for integrating Swagger with Ruby and there's also an old Ruby gem that could help as well.

So I would say that #5267 (this issue here) and #5733 (which adds the PR functionality via API) are at least a bit related (if done properly). 😉

@delvh
Copy link
Member

delvh commented Apr 29, 2023

I don't see a reason why Gitea should get bloated like that.
As already mentioned, it's probably better to integrate it through a third-party service that fails your branch protection or something similar.

@delvh delvh closed this as completed Apr 29, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/enhancement An improvement of existing functionality
Projects
None yet
Development

No branches or pull requests

5 participants