disable flaky test, lint

cmd/apply_lut.go

@@ -87,7 +87,7 @@ func applyLUTToFile(sourceFilename, lutFilename string, intensity float64, quali
 var applyLutCmd = &cobra.Command{
  Use: "apply-lut",
  Short: "Apply LUT to one or more images",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(cmd *cobra.Command, _ []string) {
  input := getFlagString(cmd, "input")
  lutFile := getFlagString(cmd, "lut")
 

cmd/calendar.go

@@ -63,7 +63,7 @@ func getModDates(input string) ([]time.Time, error) {
 var calendarView = &cobra.Command{
  Use: "calendar",
  Short: "View days in which media was captured",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(_ *cobra.Command, _ []string) {
  detectedGoPro, connectionType, err := gopro.Detect()
  if err != nil {
  cui.Error(err.Error())

cmd/export_tags.go

@@ -90,7 +90,7 @@ func extractIndividual(input, output, format string) (int, error) {
 var exportTags = &cobra.Command{
  Use: "export-tags",
  Short: "Export HiLight/other tags in video",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(cmd *cobra.Command, _ []string) {
  input := getFlagString(cmd, "input")
  format := getFlagString(cmd, "format")
  output := getFlagString(cmd, "output")

cmd/import.go

@@ -24,7 +24,7 @@ import (
 var importCmd = &cobra.Command{
  Use: "import",
  Short: "Import media",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(cmd *cobra.Command, _ []string) {
  input := getFlagString(cmd, "input")
  output := getFlagString(cmd, "output")
  camera := getFlagString(cmd, "camera")

cmd/list_devices.go

@@ -15,7 +15,7 @@ import (
 var listDevicesCmd = &cobra.Command{
  Use: "list",
  Short: "List devices available for importing",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(_ *cobra.Command, _ []string) {
  partitions, _ := disk.Partitions(false)
 
  if len(partitions) >= 1 {

cmd/merge.go

@@ -16,7 +16,7 @@ import (
 var mergeCmd = &cobra.Command{
  Use: "merge",
  Short: "Merge two or more videos together",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(cmd *cobra.Command, _ []string) {
  videoMan := videomanipulation.New()
  videos := getFlagSlice(cmd, "input")
  totalFrames := 0

cmd/update.go

@@ -11,7 +11,7 @@ import (
 var updateCmd = &cobra.Command{
  Use: "update",
  Short: "Update camera firmware",
- Run: func(cmd *cobra.Command, args []string) {
+ Run: func(cmd *cobra.Command, _ []string) {
  input := getFlagString(cmd, "input")
  camera := getFlagString(cmd, "camera")
  c, err := utils.CameraGet(camera)

pkg/dji/dji.go

@@ -215,7 +215,6 @@ func (Entrypoint) Import(params utils.ImportParams) (*utils.Result, error) {
  return nil
  },
  })
-
  if err != nil {
  inlineCounter.SetFailure(err, "")
  }

pkg/gopro/config.go

@@ -8,10 +8,10 @@ import (
 
 const parent = "gopro"
 
-func gpsMinAccuracyFromConfig() uint16 {
+func gpsMinAccuracyFromConfig() uint {
  key := fmt.Sprintf("%s.gps_accuracy", parent)
  viper.SetDefault(key, 500)
- return uint16(viper.GetUint(key))
+ return viper.GetUint(key)
 }
 
 func gpsMaxAltitudeFromConfig() float64 {

pkg/gopro/gopro.go

@@ -288,7 +288,6 @@ folderLoop:
  },
  Unsorted: true,
  })
-
  if err != nil {
  inlineCounter.SetFailure(err, "")
  }
@@ -510,7 +509,6 @@ func importFromGoProV1(params utils.ImportParams) utils.Result {
  },
  Unsorted: true,
  })
-
  if err != nil {
  inlineCounter.SetFailure(err, "")
  }

pkg/gopro/location.go

@@ -67,7 +67,7 @@ GetLocation:
 
  telems := lastEvent.ShitJson()
  for _, telem := range telems {
- if telem.Altitude > gpsMaxAltitudeFromConfig() || telem.Latitude == 0 || telem.Longitude == 0 || telem.GpsAccuracy > gpsMinAccuracyFromConfig() {
+ if telem.Altitude > gpsMaxAltitudeFromConfig() || telem.Latitude == 0 || telem.Longitude == 0 || uint(telem.GpsAccuracy) > gpsMinAccuracyFromConfig() {
  continue
  }
 

pkg/insta360/insta360.go

@@ -185,7 +185,6 @@ func (Entrypoint) Import(params utils.ImportParams) (*utils.Result, error) {
  return nil
  },
  })
-
  if err != nil {
  inlineCounter.SetFailure(err, "")
  }

pkg/utils/reversegeo_test.go

@@ -14,6 +14,8 @@ type pair struct {
 }
 
 func TestPrettyAddress(t *testing.T) {
+ t.Skip("Flaky test")
+
  expected := []pair{
  {
  Address: Location{