refactor: make Platform's receiver to pointer

suzuki-shunsuke · Jan 24, 2021 · 27e130b · 27e130b
1 parent 32369fd
commit 27e130b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/pkg/cmd/exec.go b/pkg/cmd/exec.go
@@ -80,7 +80,7 @@ func (runner *Runner) execAction(c *cli.Context) error {
 
 	var pt api.Platform
 	if p, f := platform.Get(); f {
-		pt = p
+		pt = &p
 	}
 
 	cfgReader := config.Reader{

diff --git a/pkg/cmd/post.go b/pkg/cmd/post.go
@@ -109,7 +109,7 @@ func (runner *Runner) postAction(c *cli.Context) error {
 
 	var pt api.Platform
 	if p, f := platform.Get(); f {
-		pt = p
+		pt = &p
 	}
 
 	cfgReader := config.Reader{

diff --git a/pkg/platform/platform.go b/pkg/platform/platform.go
@@ -11,7 +11,7 @@ type Platform struct {
 	Platform cienv.Platform
 }
 
-func (pt Platform) ComplementPost(opts *option.PostOptions) error {
+func (pt *Platform) ComplementPost(opts *option.PostOptions) error {
 	if opts.Org == "" {
 		opts.Org = pt.Platform.RepoOwner()
 	}
@@ -34,14 +34,14 @@ func (pt Platform) ComplementPost(opts *option.PostOptions) error {
 	return nil
 }
 
-func (pt Platform) CI() string {
+func (pt *Platform) CI() string {
 	if pt.Platform == nil {
 		return ""
 	}
 	return pt.Platform.CI()
 }
 
-func (pt Platform) ComplementExec(opts *option.ExecOptions) error {
+func (pt *Platform) ComplementExec(opts *option.ExecOptions) error {
 	if opts.Org == "" {
 		opts.Org = pt.Platform.RepoOwner()
 	}
-Original file line number
+Diff line change
@@ Expand Up / @@ -80,7 +80,7 @@ func (runner *Runner) execAction(c *cli.Context) error { @@
     	var pt api.Platform
     	if p, f := platform.Get(); f {
-    		pt = p
+    		pt = &p
     	}
     	cfgReader := config.Reader{
@@ Expand Down @@