tools: Remove gocyclo

[ValarDragon]

We can investigate re-introducing our own fork #postlaunch.
Closes #2211

I chose to leave this commented in the makefile, as a reminder that were considering re-including postlaunch. If we ever decide to remove it permanently, lets delete it then. (Though I guess it is easy enough to add back, I don't feel a need to bikeshed this so if anyones opposed, I'll delete it now)

• Targeted PR against correct branch (see CONTRIBUTING.md)

• Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

• Added entries in PENDING.md with issue #

• rereviewed Files changed in the github PR explorer

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[codecov]

Codecov Report

Merging #2228 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2228   +/-   ##
========================================
  Coverage    64.23%   64.23%           
========================================
  Files          140      140           
  Lines         8575     8575           
========================================
  Hits          5508     5508           
  Misses        2690     2690           
  Partials       377      377

[cwgoes]

utACK

You wrote remove golint in the changelog but I think you meant remove gocyclo, updated the entry