lgtm integration #390
Replies: 1 comment 8 replies
-
|
That looks interesting. I'd like to see how it works, are you using this service anywhere? |
Beta Was this translation helpful? Give feedback.
-
|
I'm +1 to add it. WDYT @quarkiverse/quarkiverse-owners ? |
Beta Was this translation helpful? Give feedback.
-
|
Obviously, just as we did with the Stale app, that would be enabled on a per-repository basis: https://github.com/quarkiverse/quarkiverse-devops/blob/main/quarkus-rabbitmq-client.tf#L48-L54 |
Beta Was this translation helpful? Give feedback.
-
|
I created a PR that enables the LGTM app for the I'll wait for the team response before merging it |
Beta Was this translation helpful? Give feedback.
-
|
Just merged it, let's see how that works for you |
Beta Was this translation helpful? Give feedback.
-
|
Thanks so much! Look forward to trying it, we already had several issues flagged. |
Beta Was this translation helpful? Give feedback.
-
@gastaldi I've been thinking of introducing lgtm for the quarkus-cxf project, particularly for automatic code reviews for PRs. Is this something you can help with, as I don't think I have access to the necessary GitHub settings? Please do share your thoughts!
Beta Was this translation helpful? Give feedback.
All reactions