Update pkg/plugin/plugin.go

benoittgt · gjenkins8 · mattfarina · commit e8bfa0eed689 · 2025-05-27T13:48:48.000-04:00
Co-authored-by: George Jenkins <gvjenkins@gmail.com> Signed-off-by: Benoit Tigeot <benoittgt@users.noreply.github.com> (cherry picked from commit dce60ad)
diff --git a/pkg/plugin/plugin.go b/pkg/plugin/plugin.go
@@ -270,7 +270,7 @@ func validatePluginData(plug *Plugin, filepath string) error {
 	}
 
 	if len(plug.Metadata.PlatformHooks) > 0 && len(plug.Metadata.Hooks) > 0 {
-		fmt.Printf("both platformHooks and hooks are set in %q. It's deprecated and will not be possible in Helm v4\n", filepath)
+		fmt.Printf("WARNING: both 'platformHooks' and 'hooks' are set in %q (this will become an error in a future Helm version)\n", filepath)
 	}
 
 	// We could also validate SemVer, executable, and other fields should we so choose.

Original file line number	Diff line number	Diff line change
`@@ -270,7 +270,7 @@ func validatePluginData(plug *Plugin, filepath string) error {`
`270`	`270`	`}`
`271`	`271`
`272`	`272`	`if len(plug.Metadata.PlatformHooks) > 0 && len(plug.Metadata.Hooks) > 0 {`
`273`		`- fmt.Printf("both platformHooks and hooks are set in %q. It's deprecated and will not be possible in Helm v4\n", filepath)`
	`273`	`+ fmt.Printf("WARNING: both 'platformHooks' and 'hooks' are set in %q (this will become an error in a future Helm version)\n", filepath)`
`274`	`274`	`}`
`275`	`275`
`276`	`276`	`// We could also validate SemVer, executable, and other fields should we so choose.`