Fix minor linting issues

internal/add/add.go

@@ -10,7 +10,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// Add checks the hosts file that the provided hosts are assigned to the ip
+// Add checks the hosts file that the provided hosts are assigned to the ip.
 func Add(cmd *cobra.Command, args []string) error {
 	hosts, err := goodhosts.NewHosts(flags.Section)
 	if err != nil {

internal/check/check.go

@@ -10,7 +10,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// Check checks the hosts file that the provided hosts are assigned to the ip
+// Check checks the hosts file that the provided hosts are assigned to the ip.
 func Check(cmd *cobra.Command, args []string) error {
 	hosts, err := goodhosts.NewHosts(flags.Section)
 	if err != nil {

internal/flags/flags.go

@@ -2,10 +2,10 @@ package flags
 
 import "github.com/spf13/cobra"
 
-// AllLines Set to true to display comments when listing
+// AllLines Set to true to display comments when listing.
 var AllLines bool
 
-// Comment Adds a comment to the entry for better identification later
+// Comment Adds a comment to the entry for better identification later.
 var Comment string
 
 // Section Is a named section with which to edit hosts in.

internal/list/list.go

@@ -9,7 +9,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// List lists all the entries in the hosts file
+// List lists all the entries in the hosts file.
 func List(cmd *cobra.Command, args []string) error {
 	hosts, err := goodhosts.NewHosts(flags.Section)
 	if err != nil {

internal/remove/remove.go

@@ -10,7 +10,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// Remove deletes a host/ip entry
+// Remove deletes a host/ip entry.
 func Remove(cmd *cobra.Command, args []string) error {
 	hosts, err := goodhosts.NewHosts(flags.Section)
 	if err != nil {

internal/removesection/removesection.go

@@ -10,7 +10,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// RemoveSection deletes a section from the hosts file
+// RemoveSection deletes a section from the hosts file.
 func RemoveSection(cmd *cobra.Command, args []string) error {
 	if flags.Section == "" {
 		return errors.New("you must provide the `--section` flag to use this command")

pkg/goodhosts/goodhosts.go

@@ -7,7 +7,7 @@ import (
 
 // NewHosts Return a new instance of “Hosts“.
 func NewHosts(sectionName string) (Hosts, error) {
-	osHostsFilePath := ""
+	var osHostsFilePath string
 
 	if os.Getenv("HOSTS_PATH") == "" {
 		osHostsFilePath = os.ExpandEnv(filepath.FromSlash(hostsFilePath))

pkg/goodhosts/hostsFile.go

@@ -83,13 +83,13 @@ func (h *Hosts) Flush() error {
 	}
 
 	if len(h.SectionLines) > 0 {
-		if len(h.Section) > 0 {
+		if h.Section != "" {
 			h.FileLines = append(h.FileLines, NewHostsLine(""), NewHostsLine(fmt.Sprintf("%s %s", sectionStart, h.Section)))
 		}
 
 		h.FileLines = append(h.FileLines, h.SectionLines...)
 
-		if len(h.Section) > 0 {
+		if h.Section != "" {
 			h.FileLines = append(h.FileLines, NewHostsLine(fmt.Sprintf("%s %s", sectionEnd, h.Section)), NewHostsLine(""))
 		}
 	}
@@ -145,7 +145,7 @@ func (h *Hosts) Has(ip, host string, forceFile bool) bool {
 	return pos != -1
 }
 
-// RemoveSection removes an entire section from the hostsfile
+// RemoveSection removes an entire section from the hostsfile.
 func (h *Hosts) RemoveSection() error {
 	if h.Section == "" {
 		return errors.New("no section provided")
@@ -191,7 +191,7 @@ func (h *Hosts) Remove(ip string, hosts ...string) error {
 				newLineRaw = fmt.Sprintf("%s %s", newLineRaw, host)
 			}
 
-			if len(line.Comment) > 0 {
+			if line.Comment != "" {
 				newLineRaw = fmt.Sprintf("%s #%s", newLineRaw, line.Comment)
 			}
 
@@ -212,7 +212,7 @@ func (h *Hosts) Remove(ip string, hosts ...string) error {
 func (h *Hosts) getHostPosition(ip, host string, forceFile bool) int {
 	checkLines := h.FileLines
 
-	if len(h.Section) > 0 && !forceFile {
+	if h.Section != "" && !forceFile {
 		checkLines = h.SectionLines
 	}
 

pkg/goodhosts/utils.go

@@ -5,7 +5,7 @@ import (
 	"strings"
 )
 
-// intemInSlice Returns true if the item string is in the provided slice or false
+// intemInSlice Returns true if the item string is in the provided slice or false.
 func itemInSlice(item string, list []string) bool {
 	for _, i := range list {
 		if i == item {
@@ -16,13 +16,13 @@ func itemInSlice(item string, list []string) bool {
 	return false
 }
 
-// buildRawLine builds a line for insertion into the hosts file
+// buildRawLine builds a line for insertion into the hosts file.
 func buildRawLine(ip, host, comment string) string {
 	output := ip
 
 	output = fmt.Sprintf("%s %s", output, host)
 
-	if len(comment) > 0 {
+	if comment != "" {
 		comment = "#" + comment
 		output = fmt.Sprintf("%s %s", output, comment)
 	}