[keycloak-dev] Introduce GolangCI to Gatekeeper repository
sthorger at redhat.com
Mon Feb 18 04:32:41 EST 2019
On Mon, 18 Feb 2019 at 10:26, Bruno Oliveira <bruno at abstractj.org> wrote:
> Thank you. Merged.
> I don't have admin rights over the repository. Could you please enable
> it here: https://golangci.com/repos/github
> On Mon, Feb 18, 2019 at 6:18 AM Stian Thorgersen <sthorger at redhat.com>
> > If it's easy to setup why not try and see? We can always re-evaluate in
> the future if benefits vs costs ratio is not correct.
> > On Wed, 13 Feb 2019 at 14:29, Bruno Oliveira <bruno at abstractj.org>
> >> Good morning,
> >> A week ago Frederic, one of our contributors, suggested the
> >> inclusion of GolangCI. For those not familiar,
> >> GolangCI can run a set of linters against the source tree. It can
> >> check if the code respects the code style, if there are vulnerabilities
> >> introduced into the codebase and also advise on best practices.
> >> Even though I think it's something nice to have, I'm afraid that
> >> GolangCI could add too much noise commenting on PRs (I'm not
> >> sure if this can be disabled).
> >> Even though is something cool, I'm 50/50 on this. I don't see the
> >> adoption from big projects like Openshift or Kubernetes.
> >> Thoughts?
> >>  - https://github.com/keycloak/keycloak-gatekeeper/pull/454
> >>  - https://golangci.com/
> >> --
> >> abstractj
> >> _______________________________________________
> >> keycloak-dev mailing list
> >> keycloak-dev at lists.jboss.org
> >> https://lists.jboss.org/mailman/listinfo/keycloak-dev
> - abstractj
More information about the keycloak-dev