Skip to content

Commit 4ee0820

Browse files
committed
fix(ci): expand get-vars job and update dependencies in orchestrator
Expanded the `get-vars` job in `orchestrator.yml` to output `npm-registry-scope` and `npm-registry-url` in addition to `cli-package-name`. The `build-and-publish` and `e2e` jobs now correctly depend on `get-vars` and consume all three outputs, ensuring proper plumbing of environment variables for the Docker build.
1 parent 070b74b commit 4ee0820

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

.github/workflows/orchestrator.yml

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,24 +36,29 @@ jobs:
3636
name: 'Lint'
3737
uses: './.github/workflows/lint.yml'
3838

39-
build-and-publish:
40-
name: 'Build and Publish CI Bundle'
41-
needs: 'lint'
42-
uses: './.github/workflows/build-and-publish.yml'
43-
with:
44-
cli-package-name: '${{ vars.CLI_PACKAGE_NAME }}'
45-
npm-registry-scope: '${{ vars.NPM_REGISTRY_SCOPE }}'
46-
npm-registry-url: '${{ vars.NPM_REGISTRY_URL }}'
47-
4839
get-vars:
4940
name: 'Get Environment Variables'
5041
runs-on: 'ubuntu-latest'
5142
environment: 'dev'
5243
outputs:
5344
cli-package-name: '${{ steps.get-vars.outputs.cli-package-name }}'
45+
npm-registry-scope: '${{ steps.get-vars.outputs.npm-registry-scope }}'
46+
npm-registry-url: '${{ steps.get-vars.outputs.npm-registry-url }}'
5447
steps:
5548
- id: 'get-vars'
56-
run: 'echo ''cli-package-name=''''${{ vars.CLI_PACKAGE_NAME }}'''''' >> "$GITHUB_OUTPUT"'
49+
run: |
50+
echo 'cli-package-name=${{ vars.CLI_PACKAGE_NAME }}' >> "$GITHUB_OUTPUT"
51+
echo 'npm-registry-scope=${{ vars.NPM_REGISTRY_SCOPE }}' >> "$GITHUB_OUTPUT"
52+
echo 'npm-registry-url=${{ vars.NPM_REGISTRY_URL }}' >> "$GITHUB_OUTPUT"
53+
54+
build-and-publish:
55+
name: 'Build and Publish CI Bundle'
56+
needs: ['lint', 'get-vars']
57+
uses: './.github/workflows/build-and-publish.yml'
58+
with:
59+
cli-package-name: '${{ needs.get-vars.outputs.cli-package-name }}'
60+
npm-registry-scope: '${{ needs.get-vars.outputs.npm-registry-scope }}'
61+
npm-registry-url: '${{ needs.get-vars.outputs.npm-registry-url }}'
5762

5863
ci:
5964
name: 'CI Checks'

0 commit comments

Comments
 (0)