[keycloak-dev] Introduce GolangCI to Gatekeeper repository

Bruno Oliveira bruno at abstractj.org
Mon Feb 18 04:24:51 EST 2019


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> wrote:
>
> 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> wrote:
>>
>> Good morning,
>>
>> A week ago Frederic, one of our contributors, suggested the
>> inclusion of GolangCI[1][2]. 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?
>>
>> [1] - https://github.com/keycloak/keycloak-gatekeeper/pull/454
>> [2] - 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 mailing list