Skip to content

Commit d26a095

Browse files
📝[Fix]: Refactor linter environment variable handling for improved clarity and security
1 parent 9df258f commit d26a095

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

‎.github/workflows/workflow.yml‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -115,15 +115,17 @@ jobs:
115115
run: |
116116
Write-Host "Loading settings for super-linter:"
117117
$settings = $env:SETTINGS | ConvertFrom-Json -AsHashtable
118+
$linter = $settings.Linter
119+
$env = $linter.env
118120
119-
foreach ($key in $settings.Linter.env.Keys) {
120-
$value = $settings.Linter.env[$key]
121+
foreach ($key in $env.Keys) {
122+
$value = $env[$key]
121123
122124
if ($value -is [bool]) {
123125
$value = $value.ToString().ToLower()
124126
}
125127
126-
Write-Host " $key = $value"
128+
Write-Host "$key = $value"
127129
128130
# Persist for following steps in this job
129131
"$key=$value" | Out-File -FilePath $env:GITHUB_ENV -Encoding utf8 -Append
@@ -132,7 +134,7 @@ jobs:
132134
- name: Lint code base
133135
uses: super-linter/super-linter@7bba2eeb89d01dc9bfd93c497477a57e72c83240 # v8.2.0
134136
env:
135-
GITHUB_TOKEN: ${{ github.token }}
137+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
136138

137139
# Runs on:
138140
# - âś… Open/Updated PR - Builds module for testing

0 commit comments

Comments
 (0)