Apply suggestions from code review

Co-authored-by: Adrien Delorme <azr@users.noreply.github.com>
5 years ago · 7ef8b290e8
parent 3fd8066a95
commit 7ef8b290e8
3 changed files with 9 additions and 15 deletions
--- a/packer-plugin-sdk/sdk-internals/communicator/none/communicator.go
+++ b/packer-plugin-sdk/sdk-internals/communicator/none/communicator.go
@ -1,8 +1,6 @@
-/*
-Package none implements the 'none' communicator. Plugin maintainers should not
-import this package directly, instead using the tooling in the
-"packer-plugin-sdk/communicator" module.
-*/
+// Package none implements the 'none' communicator. Plugin maintainers should not
+// import this package directly, instead using the tooling in the
+// "packer-plugin-sdk/communicator" module.
 package none

 import (
--- a/packer-plugin-sdk/sdk-internals/communicator/ssh/communicator.go
+++ b/packer-plugin-sdk/sdk-internals/communicator/ssh/communicator.go
@ -1,8 +1,6 @@
-/*
-Package ssh implements the SSH communicator. Plugin maintainers should not
-import this package directly, instead using the tooling in the
-"packer-plugin-sdk/communicator" module.
-*/
+// Package ssh implements the SSH communicator. Plugin maintainers should not
+// import this package directly, instead using the tooling in the
+// "packer-plugin-sdk/communicator" module.
 package ssh

 import (
--- a/packer-plugin-sdk/sdk-internals/communicator/winrm/communicator.go
+++ b/packer-plugin-sdk/sdk-internals/communicator/winrm/communicator.go
@ -1,8 +1,6 @@
-/*
-Package winrm implements the WinRM communicator. Plugin maintainers should not
-import this package directly, instead using the tooling in the
-"packer-plugin-sdk/communicator" module.
-*/
+// Package winrm implements the WinRM communicator. Plugin maintainers should not
+// import this package directly, instead using the tooling in the
+// "packer-plugin-sdk/communicator" module.
 package winrm

 import (