Merge pull request #19 from NHSDigital/mm-odin-261-commiter-does-not-come-through-as-dependabot

odin-261: dependabot commits show as committer web-flow

Author: matt-mercer

.github/workflows/pull-request.yml

@@ -101,11 +101,6 @@ jobs:
                 throw new Error(`auto-merge-dependabot not permitted for author login: ${author.login} type: ${author.type}`);
               }
 
-              let committer = commit.committer;
-              if (committer.login != 'dependabot[bot]' || committer.type != 'Bot') {
-                throw new Error(`auto-merge-dependabot not permitted for committer login: ${committer.login} type: ${committer.type}`);
-              }
-
             }
 
             if (commits.data.length>249) {

iam.tf

@@ -83,6 +83,11 @@ resource "aws_iam_role_policy" "this" {
   policy = data.aws_iam_policy_document.this.json
 }
 
+resource "aws_iam_role_policy_attachment" "logging" {
+  role       = aws_iam_role.this.name
+  policy_arn = "arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
+}
+
 resource "aws_iam_role_policy_attachment" "insights" {
   count      = var.enable_lambda_insights ? 1 : 0
   role       = aws_iam_role.this.name