Skip to content

Go Vet, Go Fmt, and Golint#225

Merged
afeld merged 1 commit intomasterfrom
cleanup
Oct 3, 2016
Merged

Go Vet, Go Fmt, and Golint#225
afeld merged 1 commit intomasterfrom
cleanup

Conversation

@jcscottiii
Copy link
Member

A cleanup patch. Since codeclimate won't run on PRs that target non-master targets, figured we need to fix that all now

Builds on #217

@codecov-io
Copy link

codecov-io commented Sep 29, 2016

Current coverage is 92.12% (diff: 75.00%)

Merging #225 into move-opencontrol-interface will increase coverage by 0.09%

@@           move-opencontrol-interface       #225   diff @@
============================================================
  Files                              30         30          
  Lines                             979        978     -1   
  Methods                             0          0          
  Messages                            0          0          
  Branches                            0          0          
============================================================
  Hits                              901        901          
+ Misses                             64         63     -1   
  Partials                           14         14          

Powered by Codecov. Last update 9cbd029...5a1890f

@jcscottiii
Copy link
Member Author

@afeld this is good. there needs to be another PR for configuring the patch parameter. this PR doesn't change any logic.

@afeld afeld changed the base branch from move-opencontrol-interface to master October 3, 2016 21:43
@afeld afeld merged commit 62d5371 into master Oct 3, 2016
@jcscottiii jcscottiii deleted the cleanup branch October 4, 2016 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants