chore: remove some lint warnings

internal/cache/file_cache.go

@@ -65,7 +65,7 @@ func (cache folderCache) Clear() {
 	if !files.IsExisting(dir) {
 		log.Warn().Msgf("Failed to find cache directory %s.", dir)
 	}
-	files.PruneDirectory(dir)
+	_ = files.PruneDirectory(dir)
 }
 
 func (cache folderCache) imagePath(checksum string, width int) string {

internal/config/file_config.go

@@ -26,9 +26,9 @@ func NewFileConfig(file string) Config {
 
 	config := fileConfig{v: v}
 
-	v.UnmarshalKey("section", &config.sections)
-	v.UnmarshalKey("image", &config.option)
-	v.UnmarshalKey("others.folders", &config.otherFolders)
+	_ = v.UnmarshalKey("section", &config.sections)
+	_ = v.UnmarshalKey("image", &config.option)
+	_ = v.UnmarshalKey("others.folders", &config.otherFolders)
 
 	log.Debug().Msgf("Config parsed: %v", config)
 

internal/export/export.go

@@ -15,8 +15,8 @@ import (
 	"github.com/waynezhang/foto/internal/utils"
 )
 
-func Export(outputPath string, minimize bool) error {
-	return export(
+func Export(outputPath string, minimize bool) {
+	export(
 		config.Shared(),
 		outputPath,
 		minimizer(minimize),
@@ -57,7 +57,7 @@ func export(
 	minimizer mm.Minimizer,
 	cache cache.Cache,
 	ctx context,
-) error {
+) {
 	sm := ysmrr.NewSpinnerManager(
 		ysmrr.WithAnimation(animations.Dots),
 	)
@@ -79,7 +79,7 @@ func export(
 	photosDirectory := files.OutputPhotosFilePath(outputPath)
 	section, err := ctx.buildIndex(cfg)
 	if err != nil {
-		ctx.cleanDirectory(outputPath)
+		_ = ctx.cleanDirectory(outputPath)
 		utils.CheckFatalError(err, "Failed to build index.")
 	}
 
@@ -99,8 +99,6 @@ func export(
 
 	spinner.Complete()
 	sm.Stop()
-
-	return nil
 }
 
 func minimizer(minimize bool) mm.Minimizer {

internal/export/export_test.go

@@ -134,8 +134,8 @@ func TestExport(t *testing.T) {
 
 	var section1 indexer.Section
 	var section2 indexer.Section
-	mapstructure.Decode(testdata.Collection1, &section1)
-	mapstructure.Decode(testdata.Collection1, &section2)
+	_ = mapstructure.Decode(testdata.Collection1, &section1)
+	_ = mapstructure.Decode(testdata.Collection1, &section2)
 	sections := []indexer.Section{section1, section2}
 
 	mockCtx := new(MockContext)
@@ -150,7 +150,7 @@ func TestExport(t *testing.T) {
 	cfg := new(MockConfig)
 	cfg.On("GetOtherFolders").Return([]string{"folder-1", "folder-2"})
 	outputPath := "test-directory"
-	export(cfg, outputPath, minimizer, cache, mockCtx)
+	_ = export(cfg, outputPath, minimizer, cache, mockCtx)
 
 	mockCtx.AssertCalled(t, "cleanDirectory", outputPath)
 	mockCtx.AssertCalled(t, "buildIndex", cfg)
@@ -164,7 +164,7 @@ func TestCleanDirectory(t *testing.T) {
 	defer os.RemoveAll(tmp)
 
 	ctx := defaultExportContext{}
-	ctx.cleanDirectory(tmp)
+	_ = ctx.cleanDirectory(tmp)
 	assert.False(t, files.IsExisting(tmp))
 }
 
@@ -174,8 +174,8 @@ func TestExportPhotos(t *testing.T) {
 
 	var section1 indexer.Section
 	var section2 indexer.Section
-	mapstructure.Decode(testdata.Collection1, &section1)
-	mapstructure.Decode(testdata.Collection1, &section2)
+	_ = mapstructure.Decode(testdata.Collection1, &section1)
+	_ = mapstructure.Decode(testdata.Collection1, &section2)
 	sections := []indexer.Section{section1, section2}
 
 	mockFunc := new(MockFunc)

internal/files/directory_test.go

@@ -16,20 +16,20 @@ func TestDirectoryManipulating(t *testing.T) {
 	assert.True(t, IsExisting(tmp))
 
 	dir1 := filepath.Join(tmp, "parent1", "child1")
-	EnsureDirectory(dir1)
+	_ = EnsureDirectory(dir1)
 	assert.True(t, IsExisting(dir1))
 
 	// no failure to create existing directory
-	EnsureParentDirectory(dir1)
+	_ = EnsureParentDirectory(dir1)
 	assert.True(t, IsExisting(dir1))
 
 	dir2Parent := filepath.Join(tmp, "parent2")
 	dir2 := filepath.Join(dir2Parent, "child2")
-	EnsureParentDirectory(dir2)
+	_ = EnsureParentDirectory(dir2)
 	assert.True(t, IsExisting(dir2Parent))
 	assert.False(t, IsExisting(dir2))
 
-	PruneDirectory(tmp)
+	_ = PruneDirectory(tmp)
 	assert.False(t, IsExisting(tmp))
 }
 

internal/images/image_test.go

@@ -23,7 +23,7 @@ func TestGetPhotoSize(t *testing.T) {
 	assert.Equal(t, testdata.TestfileHeight, size.Height)
 
 	// test against image with orientation data
-	size, err = GetPhotoSize(testdata.RotatedImageFile)
+	size, _ = GetPhotoSize(testdata.RotatedImageFile)
 	assert.Equal(t, testdata.RotatedImageWidth, size.Width)
 	assert.Equal(t, testdata.RotatedImageHeight, size.Height)
 
@@ -49,7 +49,7 @@ func TestResizeImage(t *testing.T) {
 	err = ResizeImage(testdata.Testfile, path, testdata.ThumbnailWidth)
 	assert.Nil(t, err)
 
-	checksum, err := files.Checksum(path)
+	checksum, _ := files.Checksum(path)
 	assert.Equal(t, testdata.ExpectedThubmnailChecksum, *checksum)
 }
 

internal/indexer/indexer_test.go

@@ -29,8 +29,8 @@ func TestValidSlug(t *testing.T) {
 func TestBuild(t *testing.T) {
 	var meta1 config.SectionMetadata
 	var meta2 config.SectionMetadata
-	mapstructure.Decode(testdata.Collection1, &meta1)
-	mapstructure.Decode(testdata.Collection2, &meta2)
+	_ = mapstructure.Decode(testdata.Collection1, &meta1)
+	_ = mapstructure.Decode(testdata.Collection2, &meta2)
 
 	data := []config.SectionMetadata{meta1, meta2}
 
@@ -49,8 +49,8 @@ func TestBuild(t *testing.T) {
 func TestBuildDuplicatedSlugs(t *testing.T) {
 	var meta1 config.SectionMetadata
 	var meta2 config.SectionMetadata
-	mapstructure.Decode(testdata.Collection1, &meta1)
-	mapstructure.Decode(testdata.Collection2, &meta2)
+	_ = mapstructure.Decode(testdata.Collection1, &meta1)
+	_ = mapstructure.Decode(testdata.Collection2, &meta2)
 	meta2.Slug = meta1.Slug
 
 	data := []config.SectionMetadata{meta1, meta2}