complete the recomendations from gometalinter and include in the test sequence

[zramsay]

codeclimate gets in the way for #3949 (context.TODO() is a legit function) and does not seem to provide added value over and above implementing this line. As well, AFAICT that file should be deduplicated with this one

The current workaround is to disable checking the TODO string using the config option for fixme. I can pick up this issue

[Kubuxu]

It does provide a lot of added value as:

1. it does incremental checking (important for golint)
2. in future we would like to add gometalimter to it and then filtering in the codeclimate would be really good

[zramsay]

gotcha. as long as your OK with this hack

[zramsay]

re-named the issue to reference the extension of #3949, which will come in future additional PR. feel free to assign the issue to me