Merge pull request #41 from eksrvb/bugfix/permission-check-major-comment

add permission check major comment

Author: eksrha

docs/index.md

@@ -28,14 +28,17 @@ The patching of the version only takes effect if the merged branch begins with t
 
 | SemVer | supported aliases                           | version example |
 | ------ | ------------------------------------------- | --------------- |
-| MAJOR  | `comment PR`                                | 1.2.3 => 2.0.0  |
+| MAJOR  | [comment PR`](#major-specialties)           | 1.2.3 => 2.0.0  |
 | MINOR  | feature                                     | 1.2.3 => 1.3.0  |
 | PATCH  | bugfix                                      | 1.2.3 => 1.2.4  |
 
 ![awesome-ci workflow](aci-workflow.drawio.png "awesome-ci workflow")
 
 > Hint: this tool automatically detects your environment. Supported are __Jenkins__, __GitHub Actions__ and __GitLab CI__
 
+### Major specialties
+
+To create a major release, a member of the repos with write permissions must write a comment with the content `aci=major`. awesome-ci then creates a major release.
 
 ### Requiered and optional environment variables
 

service/getBuildInfos.go

@@ -142,7 +142,9 @@ func detectIfMajor(issueNumber int) bool {
 		panic(err)
 	}
 	for _, comment := range issueComments {
-		if strings.Contains(comment.Body, "aci=major") {
+		// Must have permission in the repo to create a major version
+		// MANNEQUIN|NONE https://docs.github.com/en/graphql/reference/enums#commentauthorassociation
+		if strings.Contains(comment.Body, "aci=major") && strings.Contains("MANNEQUIN|NONE", comment.AuthorAssociation) {
 			resBool = true
 			break
 		}