Browse Source

Merge pull request #111 from digineo/golint

Add golint
Julien Laffaye 8 years ago
parent
commit
2c34dfa9b1
2 changed files with 4 additions and 4 deletions
  1. 2 0
      .travis.yml
  2. 2 4
      ftp.go

+ 2 - 0
.travis.yml

@@ -11,5 +11,7 @@ before_install:
 - sudo $TRAVIS_BUILD_DIR/.travis/prepare.sh "$FTP_SERVER"
 - sudo sysctl net.ipv6.conf.lo.disable_ipv6=0
 - go get github.com/mattn/goveralls
+- go get github.com/golang/lint/golint
 script:
 - goveralls -v
+- golint -set_exit_status $(go list ./...)

+ 2 - 4
ftp.go

@@ -131,11 +131,9 @@ func (c *ServerConn) Login(user, password string) error {
 	}
 
 	// Switch to UTF-8
-	if err := c.setUTF8(); err != nil {
-		return err
-	}
+	err = c.setUTF8()
 
-	return nil
+	return err
 }
 
 // feat issues a FEAT FTP command to list the additional commands supported by