Skip to content

Commit e116451

Browse files
Refactor Github Action per b/485167538
1 parent c40da1a commit e116451

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

.github/workflows/installations.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,9 @@ jobs:
123123
run: echo "::set-output name=val::$([[ -z $plist_secret ]] && echo "0" || echo "1")"
124124
- name: PodLibLint Installations Cron
125125
run: |
126-
export FIS_INTEGRATION_TESTS_REQUIRED=${{ steps.secrets.outputs.val }}
126+
export FIS_INTEGRATION_TESTS_REQUIRED=${STEPS_SECRETS_OUTPUTS_VAL}
127127
scripts/third_party/travis/retry.sh scripts/pod_lib_lint.rb FirebaseInstallations.podspec \
128128
--platforms=${{ matrix.target }} ${{ matrix.flags }} \
129129
--ignore-local-podspecs=FirebaseInstanceID.podspec
130+
env:
131+
STEPS_SECRETS_OUTPUTS_VAL: ${{ steps.secrets.outputs.val }}

.github/workflows/instanceid.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,5 +79,7 @@ jobs:
7979
run: echo "::set-output name=val::$([[ -z $plist_secret ]] && echo "0" || echo "1")"
8080
- name: PodLibLint InstanceID Cron
8181
run: |
82-
export FIR_IID_INTEGRATION_TESTS_REQUIRED=${{ steps.secrets.outputs.val }}
82+
export FIR_IID_INTEGRATION_TESTS_REQUIRED=${STEPS_SECRETS_OUTPUTS_VAL}
8383
FIR_IID_INTEGRATION_TESTS_REQUIRED="$plist_secret" scripts/third_party/travis/retry.sh scripts/pod_lib_lint.rb FirebaseInstanceID.podspec --platforms=${{ matrix.target }} ${{ matrix.flags }}
84+
env:
85+
STEPS_SECRETS_OUTPUTS_VAL: ${{ steps.secrets.outputs.val }}

.github/workflows/test_coverage.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
env:
4040
pr_branch: ${{ github.event.pull_request.head.ref }}
4141
run: |
42-
if [ ! -z "${{ env.METRICS_SERVICE_SECRET }}" ]; then
42+
if [ ! -z "${METRICS_SERVICE_SECRET}" ]; then
4343
./scripts/code_coverage_report/get_updated_files.sh
4444
fi
4545

0 commit comments

Comments
 (0)