Revive is fast, configurable, extensible, flexible, and beautiful linter for Go. Drop-in replacement of golint.

This is part1 of fixes for problems found for following set of rules:

• [rule.blank-imports]
  • Disallows blank imports -> Blank import should be only in a main or test package, or have a comment justifying it.
• [rule.context-as-argument]
  • context.Context should be the first argument of a function. -> By convention, context.Context should be the first parameter of a function. This rule spots function declarations that do not follow the convention.
• [rule.dot-imports]
  • Forbids . imports. -> Importing with . makes the programs much harder to understand because it is unclear whether names belong to the current package or to an imported package. More information here
• [rule.error-strings]
  • Conventions around error strings. -> Description: By convention, for better readability, error messages should not be capitalized or end with punctuation or a newline. More information here
• [rule.error-naming]
  • Naming of error variables. -> By convention, for the sake of readability, variables of type error must be named with the prefix err.
• [rule.indent-error-flow]
  • Prevents redundant else statements. -> To improve the readability of code, it is recommended to reduce the indentation as much as possible. This rule highlights redundant else-blocks that can be eliminated from the code. More information here
• [rule.errorf]
  • Should replace errors.New(fmt.Sprintf()) with fmt.Errorf() -> It is possible to get a simpler program by replacing errors.New(fmt.Sprintf()) with fmt.Errorf(). This rule spots that kind of simplification opportunities.