Skip to content
This repository has been archived by the owner on Jul 26, 2021. It is now read-only.

include static analysis tool options for Go #213

Closed
afeld opened this issue Sep 14, 2016 · 6 comments · Fixed by #338
Closed

include static analysis tool options for Go #213

afeld opened this issue Sep 14, 2016 · 6 comments · Fixed by #338

Comments

@afeld
Copy link
Contributor

afeld commented Sep 14, 2016

...because @jcscottiii finally found one! We should first make sure that it actually works and is effective, before adding it to the list.

https://github.com/HewlettPackard/gas

@NoahKunin
Copy link
Contributor

I saw that - I know we're approaching formalization on this, so CCing @dlapiduz and @cnelson so we're all on the same page (whether it's a Go app on cloud.gov or in/part of cloud.gov)

@cnelson
Copy link

cnelson commented Sep 20, 2016

I think after today's conversation, we should probably just list Go under the existing Code Climate section, as Code Climate now ships golint and govet engines.

@NoahKunin
Copy link
Contributor

@cnelson works for me! A cursory review just now indicates that's probably the best we'll get for now re: Go, but that Code Climate still isn't matching Brakeman/Hakiri/Gemnasium for Ruby on Rails (which is weird).

@afeld
Copy link
Contributor Author

afeld commented Mar 31, 2017

golint and govet

I think those are purely style, rather than having anything related to security. I could be wrong!

@jcscottiii
Copy link

if we want a swiss army knife and more, there's gometalinter (it includes Gas too)

@afeld
Copy link
Contributor Author

afeld commented Jul 15, 2017

Discussions about adding golang support to

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

Successfully merging a pull request may close this issue.

4 participants