Skip to content

Commit

Permalink
Updates from GitHub Actions Format Code workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
modernisation-platform-ci committed Nov 7, 2023
1 parent 1cffbb4 commit 65f653f
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
## A reference to the issue / Description of it.
## A reference to the issue / Description of it

{Please write here}

Expand All @@ -14,7 +14,7 @@ Please describe the tests that you ran and provide instructions to reproduce.

## Deployment Plan / Instructions

Will this deployment impact the platform and / or services on it?
Will this deployment impact the platform and / or services on it?

{Please write here}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ locals {
environment_configuration = local.environment_configurations[local.environment]
environment_configurations = {
development = {
source_accounts =[
source_accounts = [
local.environment_management.account_ids["data-platform-apps-and-tools-development"],
local.environment_management.account_ids["data-platform-development"],
local.environment_management.account_ids["data-platform-test"],
Expand All @@ -17,7 +17,7 @@ locals {
data_platform_apps_tools_account_id = local.environment_management.account_ids["data-platform-apps-and-tools-development"]
}
production = {
source_accounts =[
source_accounts = [
local.environment_management.account_ids["data-platform-production"],
local.environment_management.account_ids["data-platform-apps-and-tools-production"]
]
Expand Down
6 changes: 3 additions & 3 deletions terraform/environments/observability-platform/iam-policies.tf
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@ module "amazon_managed_prometheus_iam_policy" {

data "aws_iam_policy_document" "amazon_managed_prometheus_remote_cloudwatch" {
statement {
sid = "AllowAssumeRole"
effect = "Allow"
actions = ["sts:AssumeRole"]
sid = "AllowAssumeRole"
effect = "Allow"
actions = ["sts:AssumeRole"]
resources = formatlist("arn:aws:iam::%s:role/observability-platform", local.environment_configuration.source_accounts)
}
}
Expand Down

0 comments on commit 65f653f

Please sign in to comment.