Skip to content

Commit 03b36db

Browse files
Refactor Github Action per b/485167538
1 parent 643b737 commit 03b36db

File tree

5 files changed

+38
-16
lines changed

5 files changed

+38
-16
lines changed

.github/workflows/check-changeset.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,14 @@ jobs:
5252
- name: Print changeset checker output
5353
run: |
5454
cat << 'eof_delimiter_that_will_never_occur_in_CHANGESET_ERROR_MESSAGE'
55-
${{steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
55+
${STEPS_CHECK_CHANGESET_OUTPUTS_CHANGESET_ERROR_MESSAGE}
5656
eof_delimiter_that_will_never_occur_in_CHANGESET_ERROR_MESSAGE
57+
env:
58+
STEPS_CHECK_CHANGESET_OUTPUTS_CHANGESET_ERROR_MESSAGE: ${{steps.check-changeset.outputs.CHANGESET_ERROR_MESSAGE}}
5759
- name: Print blocking failure status
58-
run: echo "${{steps.check-changeset.outputs.BLOCKING_FAILURE}}"
60+
run: echo "${STEPS_CHECK_CHANGESET_OUTPUTS_BLOCKING_FAILURE}"
61+
env:
62+
STEPS_CHECK_CHANGESET_OUTPUTS_BLOCKING_FAILURE: ${{steps.check-changeset.outputs.BLOCKING_FAILURE}}
5963
- name: Find Comment
6064
# This commit represents v3.1.0
6165
uses: peter-evans/find-comment@3eae4d37986fb5a8592848f6a574fdf654e61f9e

.github/workflows/e2e-test.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,17 +52,20 @@ jobs:
5252
echo "export const config = $PROJECT_CONFIG; export const testAccount = $TEST_ACCOUNT" > firebase-config.js
5353
- name: Poll npm until version to test is available for install
5454
run: |
55-
echo "Polling npm for firebase@${{ github.event.client_payload.versionOrTag }}"
55+
echo "Polling npm for firebase@${GITHUB_EVENT_CLIENT_PAYLOAD_VERSIONORTAG}"
5656
node ./scripts/release/poll-npm-publish.js
5757
# run in root
5858
working-directory: '.'
5959
env:
6060
VERSION: ${{ github.event.client_payload.versionOrTag }}
61+
GITHUB_EVENT_CLIENT_PAYLOAD_VERSIONORTAG: ${{ github.event.client_payload.versionOrTag }}
6162
- name: Yarn install
6263
run: |
63-
echo "Installing firebase@${{ github.event.client_payload.versionOrTag }}"
64-
yarn add firebase@${{ github.event.client_payload.versionOrTag }}
64+
echo "Installing firebase@${GITHUB_EVENT_CLIENT_PAYLOAD_VERSIONORTAG}"
65+
yarn add firebase@${GITHUB_EVENT_CLIENT_PAYLOAD_VERSIONORTAG}
6566
yarn
67+
env:
68+
GITHUB_EVENT_CLIENT_PAYLOAD_VERSIONORTAG: ${{ github.event.client_payload.versionOrTag }}
6669
- name: Deploy "callTest" cloud function
6770
run: |
6871
pushd functions

.github/workflows/prerelease-manual-deploy.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141
- name: Yarn install
4242
run: yarn
4343
- name: Deploy prerelease
44-
run: yarn release custom -p ${{ github.event.inputs.prereleaseName }} -t ${{ github.event.inputs.npmTag }}
44+
run: yarn release custom -p ${GITHUB_EVENT_INPUTS_PRERELEASENAME} -t ${GITHUB_EVENT_INPUTS_NPMTAG}
4545
env:
4646
NPM_TOKEN_ANALYTICS: ${{secrets.NPM_TOKEN_ANALYTICS}}
4747
NPM_TOKEN_ANALYTICS_INTEROP_TYPES: ${{secrets.NPM_TOKEN_ANALYTICS_INTEROP_TYPES}}
@@ -93,4 +93,6 @@ jobs:
9393
NPM_TOKEN_APP_CHECK_COMPAT: ${{ secrets.NPM_TOKEN_APP_CHECK_COMPAT }}
9494
NPM_TOKEN_API_DOCUMENTER: ${{ secrets.NPM_TOKEN_API_DOCUMENTER }}
9595
CI: true
96-
96+
GITHUB_EVENT_INPUTS_PRERELEASENAME: ${{ github.event.inputs.prereleaseName }}
97+
GITHUB_EVENT_INPUTS_NPMTAG: ${{ github.event.inputs.npmTag }}
98+

.github/workflows/release-prod.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,16 +112,20 @@ jobs:
112112
echo "BASE_VERSION=$VERSION" >> $GITHUB_OUTPUT
113113
- name: Echo version in shell
114114
run: |
115-
echo "Base version: ${{ steps.get-version.outputs.BASE_VERSION }}"
115+
echo "Base version: ${STEPS_GET_VERSION_OUTPUTS_BASE_VERSION}"
116+
env:
117+
STEPS_GET_VERSION_OUTPUTS_BASE_VERSION: ${{ steps.get-version.outputs.BASE_VERSION }}
116118
- name: Log to release tracker
117119
# Sends release information to cloud functions endpoint of release tracker.
118120
run: |
119121
DATE=$(date +'%m/%d/%Y')
120-
BASE_VERSION=${{ steps.get-version.outputs.BASE_VERSION }}
122+
BASE_VERSION=${STEPS_GET_VERSION_OUTPUTS_BASE_VERSION}
121123
RELEASE_TRACKER_URL=${{ secrets.RELEASE_TRACKER_URL }}
122124
curl -X POST -H "Content-Type:application/json" \
123125
-d "{\"version\":\"$BASE_VERSION\",\"date\":\"$DATE\"}" \
124126
$RELEASE_TRACKER_URL/logProduction
127+
env:
128+
STEPS_GET_VERSION_OUTPUTS_BASE_VERSION: ${{ steps.get-version.outputs.BASE_VERSION }}
125129
- name: Create GitHub release
126130
env:
127131
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/release-staging.yml

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -143,19 +143,24 @@ jobs:
143143
echo "BASE_VERSION=$BASE_VERSION" >> $GITHUB_OUTPUT
144144
- name: Echo versions in shell
145145
run: |
146-
echo "Staging release ${{ steps.get-version.outputs.STAGING_VERSION }}"
147-
echo "Base version: ${{ steps.get-version.outputs.BASE_VERSION }}"
146+
echo "Staging release ${STEPS_GET_VERSION_OUTPUTS_STAGING_VERSION}"
147+
echo "Base version: ${STEPS_GET_VERSION_OUTPUTS_BASE_VERSION}"
148+
env:
149+
STEPS_GET_VERSION_OUTPUTS_STAGING_VERSION: ${{ steps.get-version.outputs.STAGING_VERSION }}
150+
STEPS_GET_VERSION_OUTPUTS_BASE_VERSION: ${{ steps.get-version.outputs.BASE_VERSION }}
148151
- name: Launch E2E tests workflow
149152
# Trigger e2e-test.yml
150153
run: |
151154
OSS_BOT_GITHUB_TOKEN=${{ secrets.OSS_BOT_GITHUB_TOKEN }}
152-
VERSION_OR_TAG=${{ steps.get-version.outputs.STAGING_VERSION }}
155+
VERSION_OR_TAG=${STEPS_GET_VERSION_OUTPUTS_STAGING_VERSION}
153156
curl -X POST \
154157
-H "Content-Type:application/json" \
155158
-H "Accept:application/vnd.github.v3+json" \
156159
-H "Authorization:Bearer $OSS_BOT_GITHUB_TOKEN" \
157160
-d "{\"event_type\":\"staging-tests\", \"client_payload\":{\"versionOrTag\":\"$VERSION_OR_TAG\"}}" \
158161
https://api.github.com/repos/firebase/firebase-js-sdk/dispatches
162+
env:
163+
STEPS_GET_VERSION_OUTPUTS_STAGING_VERSION: ${{ steps.get-version.outputs.STAGING_VERSION }}
159164
- name: Check for changes requiring a reference doc publish
160165
id: docs-check
161166
# If a diff is found (length of DIFF_CONTENTS > 0) it will write DOCS_NEEDED=true
@@ -173,11 +178,15 @@ jobs:
173178
if: ${{ always() }}
174179
run: |
175180
DATE=$(date +'%m/%d/%Y')
176-
BASE_VERSION=${{ steps.get-version.outputs.BASE_VERSION }}
177-
STAGING_VERSION=${{ steps.get-version.outputs.STAGING_VERSION }}
178-
OPERATOR=${{ github.actor }}
181+
BASE_VERSION=${STEPS_GET_VERSION_OUTPUTS_BASE_VERSION}
182+
STAGING_VERSION=${STEPS_GET_VERSION_OUTPUTS_STAGING_VERSION}
183+
OPERATOR=${GITHUB_ACTOR}
179184
RELEASE_TRACKER_URL=${{ secrets.RELEASE_TRACKER_URL }}
180-
DOCS_NEEDED=${{ steps.docs-check.outputs.DOCS_NEEDED }}
185+
DOCS_NEEDED=${STEPS_DOCS_CHECK_OUTPUTS_DOCS_NEEDED}
181186
curl -X POST -H "Content-Type:application/json" \
182187
-d "{\"version\":\"$BASE_VERSION\",\"tag\":\"$STAGING_VERSION\",\"date\":\"$DATE\",\"operator\":\"$OPERATOR\",\"docs_needed\":\"$DOCS_NEEDED\"}" \
183188
$RELEASE_TRACKER_URL/logStaging
189+
env:
190+
STEPS_GET_VERSION_OUTPUTS_BASE_VERSION: ${{ steps.get-version.outputs.BASE_VERSION }}
191+
STEPS_GET_VERSION_OUTPUTS_STAGING_VERSION: ${{ steps.get-version.outputs.STAGING_VERSION }}
192+
STEPS_DOCS_CHECK_OUTPUTS_DOCS_NEEDED: ${{ steps.docs-check.outputs.DOCS_NEEDED }}

0 commit comments

Comments
 (0)