Skip to content

Commit cfb8410

Browse files
committed
fix: GitHub Actionsワークフローのshellcheck警告を修正
Made-with: Cursor
1 parent 1d0f110 commit cfb8410

File tree

3 files changed

+42
-34
lines changed

3 files changed

+42
-34
lines changed

.github/workflows/deploy-app.yaml

Lines changed: 28 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,9 @@ jobs:
5151
exit 1
5252
fi
5353
54-
echo "デプロイするプラットフォーム: ${platforms[@]}"
54+
echo "デプロイするプラットフォーム: ${platforms[*]}"
5555
for platform in "${platforms[@]}"; do
56-
echo "deploy-${platform}=true" >> $GITHUB_OUTPUT
56+
echo "deploy-${platform}=true" >> "$GITHUB_OUTPUT"
5757
done
5858
5959
build-ios:
@@ -88,8 +88,9 @@ jobs:
8888
- name: Set environment variables
8989
run: |
9090
for line in $(mise env --redacted --dotenv); do
91-
echo $line >> "$GITHUB_ENV"
92-
echo "::add-mask::${line#*=}"
91+
echo "$line" >> "$GITHUB_ENV"
92+
mask_value="${line#*=}"
93+
echo "::add-mask::${mask_value}"
9394
done
9495
9596
# https://github.com/maxim-lobanov/setup-xcode
@@ -121,9 +122,9 @@ jobs:
121122
122123
- name: Extract App Store Connect API Key
123124
run: |
124-
mkdir ~/.private_keys
125+
mkdir -p "$HOME/.private_keys"
125126
echo "$APP_STORE_CONNECT_API_KEY_BASE64" | base64 -d \
126-
> ~/.private_keys/AuthKey_$APP_STORE_CONNECT_API_KEY_ID.p8
127+
> "$HOME/.private_keys/AuthKey_${APP_STORE_CONNECT_API_KEY_ID}.p8"
127128
128129
- name: Create XCArchive
129130
working-directory: app
@@ -138,8 +139,8 @@ jobs:
138139
CODE_SIGN_STYLE=Automatic \
139140
DEVELOPMENT_TEAM=CPL7H8SHVM \
140141
-allowProvisioningUpdates \
141-
-authenticationKeyIssuerID $APP_STORE_CONNECT_API_ISSUER_ID \
142-
-authenticationKeyID $APP_STORE_CONNECT_API_KEY_ID \
142+
-authenticationKeyIssuerID "$APP_STORE_CONNECT_API_ISSUER_ID" \
143+
-authenticationKeyID "$APP_STORE_CONNECT_API_KEY_ID" \
143144
-authenticationKeyPath "$HOME/.private_keys/AuthKey_$APP_STORE_CONNECT_API_KEY_ID.p8" \
144145
| xcbeautify --renderer github-actions
145146
@@ -151,8 +152,8 @@ jobs:
151152
-exportOptionsPlist ios/ExportOptions.plist \
152153
-exportPath . \
153154
-allowProvisioningUpdates \
154-
-authenticationKeyIssuerID $APP_STORE_CONNECT_API_ISSUER_ID \
155-
-authenticationKeyID $APP_STORE_CONNECT_API_KEY_ID \
155+
-authenticationKeyIssuerID "$APP_STORE_CONNECT_API_ISSUER_ID" \
156+
-authenticationKeyID "$APP_STORE_CONNECT_API_KEY_ID" \
156157
-authenticationKeyPath "$HOME/.private_keys/AuthKey_$APP_STORE_CONNECT_API_KEY_ID.p8" \
157158
| xcbeautify --renderer github-actions
158159
echo "--------------------------------"
@@ -214,24 +215,25 @@ jobs:
214215
- name: Set environment variables
215216
run: |
216217
for line in $(mise env --redacted --dotenv); do
217-
echo $line >> "$GITHUB_ENV"
218-
echo "::add-mask::${line#*=}"
218+
echo "$line" >> "$GITHUB_ENV"
219+
mask_value="${line#*=}"
220+
echo "::add-mask::${mask_value}"
219221
done
220222
221223
- name: Extract App Store Connect API Key
222224
run: |
223-
mkdir ~/.private_keys
225+
mkdir -p "$HOME/.private_keys"
224226
echo "$APP_STORE_CONNECT_API_KEY_BASE64" | base64 -d \
225-
> ~/.private_keys/AuthKey_$APP_STORE_CONNECT_API_KEY_ID.p8
227+
> "$HOME/.private_keys/AuthKey_${APP_STORE_CONNECT_API_KEY_ID}.p8"
226228
227229
- name: Upload Ipa to App Store Connect
228230
run: |
229231
xcrun altool \
230232
--upload-app \
231233
-f build/eqmonitor.ipa \
232234
--type ios \
233-
--apiIssuer $APP_STORE_CONNECT_API_ISSUER_ID \
234-
--apiKey $APP_STORE_CONNECT_API_KEY_ID
235+
--apiIssuer "$APP_STORE_CONNECT_API_ISSUER_ID" \
236+
--apiKey "$APP_STORE_CONNECT_API_KEY_ID"
235237
236238
build-android:
237239
needs: define-matrix
@@ -262,8 +264,9 @@ jobs:
262264
- name: Set environment variables
263265
run: |
264266
for line in $(mise env --redacted --dotenv); do
265-
echo $line >> "$GITHUB_ENV"
266-
echo "::add-mask::${line#*=}"
267+
echo "$line" >> "$GITHUB_ENV"
268+
mask_value="${line#*=}"
269+
echo "::add-mask::${mask_value}"
267270
done
268271
269272
- name: Accept Android License
@@ -279,7 +282,7 @@ jobs:
279282
echo "$KEY_PROPERTIES" | base64 -d > app/android/key.properties
280283
281284
BUILD_NUMBER=$((LATEST_BUILD_NUMBER + 1))
282-
echo "BUILD_NUMBER=${BUILD_NUMBER}" >> $GITHUB_OUTPUT
285+
echo "BUILD_NUMBER=${BUILD_NUMBER}" >> "$GITHUB_OUTPUT"
283286
echo "Build Number set to ${BUILD_NUMBER}."
284287
285288
- name: Extract environment variables
@@ -299,7 +302,7 @@ jobs:
299302
flutter build appbundle \
300303
--release \
301304
--no-pub \
302-
--build-number=${BUILD_NUMBER} \
305+
--build-number="${BUILD_NUMBER}" \
303306
--dart-define-from-file=../environment/.env.prod
304307
305308
- name: Copy AAB to expected location
@@ -347,8 +350,9 @@ jobs:
347350
- name: Set environment variables
348351
run: |
349352
for line in $(mise env --redacted --dotenv); do
350-
echo $line >> "$GITHUB_ENV"
351-
echo "::add-mask::${line#*=}"
353+
echo "$line" >> "$GITHUB_ENV"
354+
mask_value="${line#*=}"
355+
echo "::add-mask::${mask_value}"
352356
done
353357
354358
- name: Download artifact aab
@@ -374,11 +378,11 @@ jobs:
374378
run: |
375379
# LAST TAG
376380
LAST_TAG=$(git describe --tags --abbrev=0)
377-
git log $LAST_TAG..HEAD --pretty=format:"- [%ad] %an: %s %h" >> changelog.tmp
381+
git log "${LAST_TAG}..HEAD" --pretty=format:"- [%ad] %an: %s %h" >> changelog.tmp
378382
379383
# 最大2000文字
380384
head -c 2000 changelog.tmp > changelog.txt
381-
if [ $(wc -c < changelog.txt) -gt 2000 ]; then
385+
if [ "$(wc -c < changelog.txt)" -gt 2000 ]; then
382386
echo '...' >> changelog.txt
383387
fi
384388

.github/workflows/deploy-terraform-googe-cloud.yaml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
2424
# JSONの配列に変換し、outputsに設定
2525
targets=$(echo "$dirs" | jq -R . | jq -sc .)
26-
echo "targets=${targets}" >> $GITHUB_OUTPUT
26+
echo "targets=${targets}" >> "$GITHUB_OUTPUT"
2727
2828
lint:
2929
name: Terraform Lint (${{ matrix.target }})
@@ -56,8 +56,9 @@ jobs:
5656
- name: Set environment variables
5757
run: |
5858
for line in $(mise env --redacted --dotenv); do
59-
echo $line >> "$GITHUB_ENV"
60-
echo "::add-mask::${line#*=}"
59+
echo "$line" >> "$GITHUB_ENV"
60+
mask_value="${line#*=}"
61+
echo "::add-mask::${mask_value}"
6162
done
6263
6364
- name: Extract backend.tfbackend
@@ -128,8 +129,9 @@ jobs:
128129
- name: Set environment variables
129130
run: |
130131
for line in $(mise env --redacted --dotenv); do
131-
echo $line >> "$GITHUB_ENV"
132-
echo "::add-mask::${line#*=}"
132+
echo "$line" >> "$GITHUB_ENV"
133+
mask_value="${line#*=}"
134+
echo "::add-mask::${mask_value}"
133135
done
134136
135137
- name: Change directory

.github/workflows/plan-terraform-google-cloud.yaml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
2424
# JSONの配列に変換し、outputsに設定
2525
targets=$(echo "$dirs" | jq -R . | jq -sc .)
26-
echo "targets=${targets}" >> $GITHUB_OUTPUT
26+
echo "targets=${targets}" >> "$GITHUB_OUTPUT"
2727
2828
lint:
2929
name: Terraform Lint (${{ matrix.target }})
@@ -56,8 +56,9 @@ jobs:
5656
- name: Set environment variables
5757
run: |
5858
for line in $(mise env --redacted --dotenv); do
59-
echo $line >> "$GITHUB_ENV"
60-
echo "::add-mask::${line#*=}"
59+
echo "$line" >> "$GITHUB_ENV"
60+
mask_value="${line#*=}"
61+
echo "::add-mask::${mask_value}"
6162
done
6263
6364
- name: Extract backend.tfbackend
@@ -123,8 +124,9 @@ jobs:
123124
- name: Set environment variables
124125
run: |
125126
for line in $(mise env --redacted --dotenv); do
126-
echo $line >> "$GITHUB_ENV"
127-
echo "::add-mask::${line#*=}"
127+
echo "$line" >> "$GITHUB_ENV"
128+
mask_value="${line#*=}"
129+
echo "::add-mask::${mask_value}"
128130
done
129131
130132
- name: Change directory

0 commit comments

Comments
 (0)