search

adlio / trello

Trello API wrapper for Go
MIT License
220 stars 71 forks source link

Golint #32

Closed jtsaito closed 5 years ago

jtsaito commented 5 years ago

Following up on an earlier comment, this PR adds comments to all public members. This is the first of two or three steps towards satisfying golint and govet.

codecov-io commented 5 years ago

Codecov Report

Merging #32 into master will not change coverage. The diff coverage is 75%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   71.84%   71.84%           
=======================================
  Files          19       19           
  Lines         760      760           
=======================================
  Hits          546      546           
  Misses        180      180           
  Partials       34       34
Impacted Files Coverage Δ
label.go 100% <ø> (ø) :arrow_up:
search.go 100% <ø> (ø) :arrow_up:
action-collection.go 100% <ø> (ø) :arrow_up:
list-duration-sort.go 100% <ø> (ø) :arrow_up:
errors.go 63.15% <ø> (ø) :arrow_up:
list-duration.go 83.33% <ø> (ø) :arrow_up:
member.go 100% <ø> (ø) :arrow_up:
arguments.go 100% <ø> (ø) :arrow_up:
list.go 100% <ø> (ø) :arrow_up:
custom-fields.go 100% <ø> (ø) :arrow_up:
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d090822...9d1f780. Read the comment docs.

jtsaito commented 5 years ago

Closing in favour of https://github.com/adlio/trello/pull/33.