fix review issues

Author: kayman-mk

.gitignore

@@ -1,6 +1,8 @@
 # aws_iam_policy_attachment_exclusive_attachment
 
-Consider alternative resources to `aws_iam_policy_attachment`.
+This rule checks whether the `aws_iam_policy_attachment` resource is used.
+
+The `aws_iam_policy_attachment` resource creates exclusive attachments for IAM policies. Within the entire AWS account, all users, roles, and groups that a single policy is attached to must be specified by a single aws_iam_policy_attachment resource. 
 
 ## Configuration
 
@@ -23,12 +25,12 @@ $ tflint
 1 issue(s) found:
 Warning: Consider aws_iam_role_policy_attachment, aws_iam_user_policy_attachment, or aws_iam_group_policy_attachment instead. (aws_iam_policy_attachment_has_alternatives)
   on template.tf line 2:
-   2:   name "test_attachment"
+   2:  resource "aws_iam_policy_attachment" "attachment" {
 ```
 
 ## Why
 
-The `aws_iam_policy_attachment` resource creates exclusive attachments of IAM policies. Across the entire AWS account, all the users/roles/groups to which a single policy is attached must be declared by a single `aws_iam_policy_attachment` resource. This means that even any users/roles/groups that have the attached policy via any other mechanism (including other Terraform resources) will have that attached policy revoked by this resource. https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_policy_attachment
+The [`aws_iam_policy_attachment`](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_policy_attachment) resource creates exclusive attachments of IAM policies. Across the entire AWS account, all the users/roles/groups to which a single policy is attached must be declared by a single `aws_iam_policy_attachment` resource. This means that even any users/roles/groups that have the attached policy via any other mechanism (including other Terraform resources) will have that attached policy revoked by this resource.
 
 ## How To Fix
 

docs/rules/aws_iam_policy_attachment_exclusive_attachment.md

@@ -52,15 +52,10 @@ func (r *AwsIAMPolicyAttachmentExclusiveAttachmentRule) Check(runner tflint.Runn
 	}
 
 	for _, resource := range resources.Blocks {
-		attribute, exists := resource.Body.Attributes[r.attributeName]
-		if !exists {
-			continue
-		}
-
 		runner.EmitIssue(
 			r,
-			"Consider aws_iam_role_policy_attachment, aws_iam_user_policy_attachment, or aws_iam_group_policy_attachment instead.",
-			attribute.Expr.Range(),
+			"Within the entire AWS account, all users, roles, and groups that a single policy is attached to must be specified by a single aws_iam_policy_attachment resource. Consider aws_iam_role_policy_attachment, aws_iam_user_policy_attachment, or aws_iam_group_policy_attachment instead.",
+			resource.DefRange,
 		)
 
 		if err != nil {

rules/aws_iam_policy_attachment_exclusive_attachment.go

@@ -30,8 +30,8 @@ resource "aws_iam_policy_attachment" "attachment" {
 					Message: "Consider aws_iam_role_policy_attachment, aws_iam_user_policy_attachment, or aws_iam_group_policy_attachment instead.",
 					Range: hcl.Range{
 						Filename: "resource.tf",
-						Start:    hcl.Pos{Line: 3, Column: 9},
-						End:      hcl.Pos{Line: 3, Column: 26},
+						Start:    hcl.Pos{Line: 2, Column: 1},
+						End:      hcl.Pos{Line: 2, Column: 50},
 					},
 				},
 			},