search

IBM / ubiquity

Ubiquity
Apache License 2.0
90 stars 26 forks source link

Apply gofmt fixes #209

Closed shay-berman closed 6 years ago

shay-berman commented 6 years ago

Align the code with gofmt. We carry many gofmt issues, and now is the time to fix them all. We should make sure every Ubiquity developer set automatic gofmt in its IDE to prevent pushing formatting issues in the future.

We should keep go report in high score as much as possible.

This change is Reviewable

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.01%) to 52.769% when pulling 4ec6be13fdbaf4ab398e99c6efc4d197e2092d69 on fix/gofmt_alignment into f05aee3b505045aacb447c637a8ade80af7e016d on dev.

olgashtivelman commented 6 years ago

Reviewed 57 of 57 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

olgashtivelman commented 6 years ago
:lgtm:

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable