fix(gazelle): include file path in manifest parsing error message (bazel-contrib#579)

mattem · web-flow · commit a876fcab1892 · 2021-12-16T06:25:42.000-08:00
diff --git a/gazelle/configure.go b/gazelle/configure.go
@@ -154,11 +154,11 @@ func (py *Configurer) loadGazelleManifest(gazelleManifestPath string) (*manifest
 		if os.IsNotExist(err) {
 			return nil, nil
 		}
-		return nil, fmt.Errorf("failed to load Gazelle manifest: %w", err)
+		return nil, fmt.Errorf("failed to load Gazelle manifest at %q: %w", gazelleManifestPath, err)
 	}
 	manifestFile := new(manifest.File)
 	if err := manifestFile.Decode(gazelleManifestPath); err != nil {
-		return nil, fmt.Errorf("failed to load Gazelle manifest: %w", err)
+		return nil, fmt.Errorf("failed to load Gazelle manifest at %q: %w", gazelleManifestPath, err)
 	}
 	return manifestFile.Manifest, nil
 }

Original file line number	Diff line number	Diff line change
`@@ -154,11 +154,11 @@ func (py Configurer) loadGazelleManifest(gazelleManifestPath string) (manifest`
`154`	`154`	`if os.IsNotExist(err) {`
`155`	`155`	`return nil, nil`
`156`	`156`	`}`
`157`		`- return nil, fmt.Errorf("failed to load Gazelle manifest: %w", err)`
	`157`	`+ return nil, fmt.Errorf("failed to load Gazelle manifest at %q: %w", gazelleManifestPath, err)`
`158`	`158`	`}`
`159`	`159`	`manifestFile := new(manifest.File)`
`160`	`160`	`if err := manifestFile.Decode(gazelleManifestPath); err != nil {`
`161`		`- return nil, fmt.Errorf("failed to load Gazelle manifest: %w", err)`
	`161`	`+ return nil, fmt.Errorf("failed to load Gazelle manifest at %q: %w", gazelleManifestPath, err)`
`162`	`162`	`}`
`163`	`163`	`return manifestFile.Manifest, nil`
`164`	`164`	`}`