fix: github init settings

Author: coolapso

cmd/utils.go

@@ -12,28 +12,29 @@ import (
 )
 
 // This function could use a better name
+// This function could use some tests, but ghactions doesn't let overide env variables
 func initGHSettings() (err error) {
-	token, err = getGHToken()
-	if err != nil {
-		return err
+	if repository == "" {
+		repository, err = getRepository()
+		if err != nil {
+			return err
+		}
 	}
 
-	if prNumber == 0 {
-		if !githubAction() {
-			return errMissingPRNum
-		}
+	if !githubAction() && prNumber == 0 {
+		return errMissingPRNum
+	}
 
+	if pullRequest() {
 		prNumber, err = getPRNumber()
 		if err != nil {
 			return fmt.Errorf("failed to get pull request number from github context GITHUB_REF_NAME, %v", err)
 		}
 	}
 
-	if repository == "" {
-		repository, err = getRepository()
-		if err != nil {
-			return err
-		}
+	token, err = getGHToken()
+	if err != nil {
+		return err
 	}
 
 	return nil
@@ -74,7 +75,7 @@ func githubAction() bool {
 }
 
 func pullRequest() bool {
-	if v := os.Getenv("GITHUB_EVENT_TYPE"); v == "pull_request" {
+	if v := os.Getenv("GITHUB_EVENT_NAME"); v == "pull_request" {
 		return true
 	}
 

cmd/utils_test.go

@@ -132,7 +132,20 @@ func TestSplitOwnerRepo(t *testing.T) {
 
 func TestPullRequest(t *testing.T) {
 	t.Run("Is pull request", func(t *testing.T) {
-		_ = os.Setenv("GITHUB_EVENT_TYPE", "pull_request")
+		if _, ok := os.LookupEnv("GITHUB_ACTIONS"); !ok {
+			_ = os.Setenv("GITHUB_EVENT_NAME", "pull_request")
+			defer os.Unsetenv("GITHUB_EVENT_NAME")
+		}
+
+		var eventTemp string
+		if _, ok := os.LookupEnv("GITHUB_ACTIONS"); ok {
+			if v := os.Getenv("GITHUB_EVENT_NAME"); v != "pull_request" {
+				eventTemp = v
+				os.Setenv("GITHUB_EVENT_NAME", "pull_request")
+				defer func() { os.Setenv("GITHUB_EVENT_NAME", eventTemp) }()
+			}
+		}
+
 		want := true
 		got := pullRequest()
 		if got != want {
@@ -141,7 +154,7 @@ func TestPullRequest(t *testing.T) {
 	})
 
 	t.Run("Is not pull request", func(t *testing.T) {
-		_ = os.Setenv("GITHUB_EVENT_TYPE", "push")
+		_ = os.Setenv("GITHUB_EVENT_NAME", "push")
 		want := false
 		got := pullRequest()
 		if got != want {