Skip to content

Commit bdb49ca

Browse files
chore(build): add tflint exceptions and dependency updates
- add terraform plugin to .tflint.hcl for terraform_required_providers rule - disable terraform_required_providers rule (tracked by issue #106) - update prevent_destroy rule comment with issue #104 link - update werkzeug 3.1.4 to 3.1.5 (PR #93) - update actions/setup-node to v4.3.0 SHA (PR #89) 🔧 - Generated by Copilot
1 parent a42fdf5 commit bdb49ca

File tree

5 files changed

+22
-7
lines changed

5 files changed

+22
-7
lines changed

.github/workflows/cluster-test-terraform.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -337,7 +337,7 @@ jobs:
337337
run: exit 1
338338

339339
- name: Setup Node.js for test processing
340-
uses: actions/setup-node@b9b25d45f70a5d94d88496aa4896bf9ed8f49b67
340+
uses: actions/setup-node@65d868f8d4d85d7d4abb7de0875cde3fcc8798f5 # v4.3.0
341341
with:
342342
node-version: '22.x'
343343
if: steps.workingDirProcessor.outputs.tfTestsExist == 'true'

.github/workflows/docs-automation.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,7 @@ jobs:
319319
uses: actions/checkout@0c366fd6a839edf440554fa01a7085ccba70ac98 # v4.1.7
320320

321321
- name: Setup Node.js
322-
uses: actions/setup-node@b9b25d45f70a5d94d88496aa4896bf9ed8f49b67 # v4.1.1
322+
uses: actions/setup-node@65d868f8d4d85d7d4abb7de0875cde3fcc8798f5 # v4.3.0
323323
with:
324324
node-version: '18'
325325

@@ -369,7 +369,7 @@ jobs:
369369
uses: actions/checkout@0c366fd6a839edf440554fa01a7085ccba70ac98 # v4.1.7
370370

371371
- name: Setup Node.js
372-
uses: actions/setup-node@b9b25d45f70a5d94d88496aa4896bf9ed8f49b67 # v4.1.1
372+
uses: actions/setup-node@65d868f8d4d85d7d4abb7de0875cde3fcc8798f5 # v4.3.0
373373
with:
374374
node-version: '18'
375375

.github/workflows/pages-deploy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,7 @@ jobs:
143143
uses: actions/configure-pages@d5606572c479bee637007364c6b4800ac4fc8573
144144

145145
- name: Setup Node.js
146-
uses: actions/setup-node@b9b25d45f70a5d94d88496aa4896bf9ed8f49b67
146+
uses: actions/setup-node@65d868f8d4d85d7d4abb7de0875cde3fcc8798f5 # v4.3.0
147147
with:
148148
node-version: '18'
149149
cache: 'npm'

.tflint.hcl

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,24 @@ plugin "azurerm" {
77
source = "github.com/terraform-linters/tflint-ruleset-azurerm"
88
}
99

10+
plugin "terraform" {
11+
enabled = true
12+
version = "0.13.0"
13+
source = "github.com/terraform-linters/tflint-ruleset-terraform"
14+
}
15+
1016
# Temporarily disabled rules
11-
# TODO: Re-enable and fix - tracked by GitHub issue
12-
# See: https://github.com/terraform-linters/tflint-ruleset-azurerm/blob/v0.30.0/docs/rules/azurerm_resources_missing_prevent_destroy.md
17+
18+
# TODO: Re-enable after adding lifecycle prevent_destroy blocks
19+
# Tracked by: https://github.com/microsoft/edge-ai/issues/104
20+
# Reference: https://github.com/terraform-linters/tflint-ruleset-azurerm/blob/v0.30.0/docs/rules/azurerm_resources_missing_prevent_destroy.md
1321
rule "azurerm_resources_missing_prevent_destroy" {
1422
enabled = false
1523
}
24+
25+
# TODO: Re-enable after adding provider version constraints to internal modules
26+
# Tracked by: https://github.com/microsoft/edge-ai/issues/106
27+
# Reference: https://github.com/terraform-linters/tflint-ruleset-terraform/blob/v0.13.0/docs/rules/terraform_required_providers.md
28+
rule "terraform_required_providers" {
29+
enabled = false
30+
}

src/500-application/506-ros2-connector/services/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
# Web framework for health checks
55
flask==3.0.3
6-
werkzeug==3.1.4
6+
werkzeug==3.1.5
77

88
# Data processing and utilities
99
numpy>=1.26.0

0 commit comments

Comments
 (0)