Partially revert "Add golangci.yml config enabling several other lint…

…ers" This reverts commit a72cf40 which also included fixes for some of the issues raised by golangci-lint. They will be addressed in a separate PR. Signed-off-by: Radoslav Dimitrov <[email protected]>
theupdateframework · Mar 18, 2022 · f12e6d1 · f12e6d1
1 parent 24697c4
commit f12e6d1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/local_store.go b/local_store.go
@@ -315,8 +315,11 @@ func (f *fileSystemStore) Commit(consistentSnapshot bool, versions map[string]in
 			return err
 		}
 		if !info.IsDir() && isTarget(rel) && needsRemoval(rel) {
-			// TODO: log / handle error or remove empty directory
-			_ = os.Remove(path)
+			//lint:ignore SA9003 empty branch
+			if err := os.Remove(path); err != nil {
+				// TODO: log / handle error
+			}
+			// TODO: remove empty directory
 		}
 		return nil
 	}

diff --git a/verify/db_test.go b/verify/db_test.go
@@ -36,7 +36,7 @@ func TestDelegationsVerifier(t *testing.T) {
 		{
 			testName: "invalid keys",
 			delegations: &data.Delegations{Keys: map[string]*data.PublicKey{
-				"a": {Type: data.KeySchemeEd25519},
+				"a": &data.PublicKey{Type: data.KeySchemeEd25519},
 			}},
 			initErr: ErrWrongID{},
 		},