remove unused variable (#30)

* remove unused variable

* adjust lint settings

* attempt to figure out which line is causing the linter error

Author: muir

.github/workflows/golangci-lint.yml

@@ -15,6 +15,10 @@ jobs:
           # Optional: working directory, useful for monorepos
           # working-directory: somedir
           output: checkstyle
+          print-issued-lines: true
+          format: colored-line-number
+          print-linter-name: true
+          sort-results: true
 
           # Optional: golangci-lint command line arguments.
           # args: --issues-exit-code=0

.golangci.yaml

@@ -86,6 +86,7 @@ linters:
     - govet
     - exportloopref
     - errcheck
+    - exhaustive
   enable-all: false
   disable:
     - maligned
@@ -159,7 +160,7 @@ linters-settings:
     # indicates that switch statements are to be considered exhaustive if a
     # 'default' case is present, even if all enum members aren't listed in the
     # switch
-    default-signifies-exhaustive: false
+    default-signifies-exhaustive: true
 
   exhaustivestruct:
     # Struct Patterns is list of expressions to match struct packages and names

include.go

@@ -159,7 +159,6 @@ func computeDependenciesAndInclusion(funcs []*provider, initF *provider) error {
 	}
 
 	// Attempt to eliminate providers
-	postCheck := make([]*provider, 0, len(funcs))
 	for _, fm := range proposeEliminations(funcs) {
 		if fm.d.excluded != nil {
 			continue
@@ -172,8 +171,6 @@ func computeDependenciesAndInclusion(funcs []*provider, initF *provider) error {
 		tryWithout(fm)
 	}
 
-	eliminateUnused(postCheck)
-
 	debugln("final set of functions")
 	for _, fm := range funcs {
 		if fm.d.excluded == nil {