Page MenuHomePhabricator

Add linter and unit test engine for Go(lang)
Open, WishlistPublic

Description

Currently for Go(lang) there's a test result parser in GoTestResultParser.php. It'll be good to extend unit test engine to support the following:

  • coverage report

Also following are popular golang linters to have:

  • gofmt
  • golint
  • go vet

Event Timeline

roylou created this task.Jan 5 2015, 10:03 AM
roylou raised the priority of this task from to Needs Triage.
roylou updated the task description. (Show Details)
roylou added a project: Lint.
roylou added a subscriber: roylou.
chad triaged this task as Wishlist priority.Jan 5 2015, 2:58 PM
roylou added a subscriber: chad.Jan 14 2015, 6:49 AM

Hi @chad, we're working on a a project using golang and phabricator. I can submit a diff to for those functionalities. Please let me know if it sounds good to you.

What exactly are you looking for? There is an existing golang linter which wraps go lint.

Oh I didn't notice there's an existing ArcanistGoLintLinter. Thanks for pointing out. This is what I think missing:

  • A linter that wraps go vet
  • A linter that warps gofmt
  • Add coverage support in ArcanistGoTestResultParser

Not sure where this is going but I believe it's still on hold. I've release https://github.com/kalbasit/arcanist-go that I'm using personally and also at work. @joshuaspence and @chad please let me know when the Phabricator team is ready to accept these patches and I'll update all of my differentials.

ox added a subscriber: ox.Jul 30 2015, 2:03 PM
epriestley moved this task from Backlog to Bindings on the Lint board.Dec 22 2015, 12:47 PM