Skip to content
This repository has been archived by the owner on Sep 1, 2020. It is now read-only.

Fix errors reported by GolangCI #91

Merged
merged 2 commits into from
Oct 1, 2018
Merged

Fix errors reported by GolangCI #91

merged 2 commits into from
Oct 1, 2018

Conversation

Ullaakut
Copy link
Owner

@Ullaakut Ullaakut commented Oct 1, 2018

Goal of this PR

Improve code quality and fix potential errors by using GolangCI's helpful report.

This PR fixes the following errors:

screenshot 2018-10-01 at 19 36 44

screenshot 2018-10-01 at 19 36 51

screenshot 2018-10-01 at 19 37 00

@Ullaakut Ullaakut added this to the Beta version milestone Oct 1, 2018
@Ullaakut Ullaakut self-assigned this Oct 1, 2018
@coveralls
Copy link

coveralls commented Oct 1, 2018 •
edited
Loading

Pull Request Test Coverage Report for Build 116

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%
Totals Coverage Status
Change from base Build 107: 0.0%
Covered Lines: 289
Relevant Lines: 289
💛 - Coveralls

@Ullaakut Ullaakut merged commit e8c70d5 into master Oct 1, 2018
@Ullaakut Ullaakut deleted the fix-golangci-errors branch October 1, 2018 17:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@Ullaakut Ullaakut

Labels
Backend Improvement
Projects
None yet
Milestone
Beta version
Development

Successfully merging this pull request may close these issues.
2 participants
@Ullaakut @coveralls