Skip to content

Commit

Permalink
fix pacmak integ test
Browse files Browse the repository at this point in the history
  • Loading branch information
mrgrain committed Nov 15, 2024
1 parent d3c7810 commit 249853b
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -383,6 +383,7 @@ jobs:
fail-fast: false
matrix:
rosetta:
- latest
- 5.4.x
- 5.5.x
- 5.6.x
Expand Down Expand Up @@ -419,15 +420,11 @@ jobs:
run: sudo apt install -y python3-venv
# Show time!
- name: Prepare Work Tree
# 1) Move the locally build version of jsii-rosetta somewhere else
# 2) Remove @jsii/integ-test because it messed up version dependencies and is not needed
# 3) Install aws-cdk-lib, and all locally build packages minus jsii-rosetta & @jsii/integ-test,
# and the specific version of jsii-rosetta
# This ensures we are running jsii-pacmak with the correct peer-dependency
# 4) Print the jsii-rosetta version for confirmation
# 1) Install aws-cdk-lib,
# all locally build packages,
# the specific version of jsii-rosetta.
# 2) Print the jsii-rosetta version for confirmation
run: |-
mv ${{ runner.temp }}/release-package/js/jsii-rosetta.tgz ./jsii-rosetta.tgz
rm ${{ runner.temp }}/release-package/private/@jsii-integ-test.tgz
npm install --no-save --omit=dev \
aws-cdk-lib@2 \
constructs@10 \
Expand All @@ -444,7 +441,7 @@ jobs:
./node_modules/.bin/jsii-pacmak --no-parallel ./node_modules/aws-cdk-lib
# Upload artifact only on main and for latest rosetta
- name: 'Upload Artifact: integtest_aws-cdk-lib'
if: github.ref == 'ref/head/main' && matrix.rosetta == '5.4.x'
if: github.ref == 'ref/head/main' && matrix.rosetta == 'latest'
uses: actions/upload-artifact@v4
with:
name: integtest_aws-cdk-lib
Expand Down

0 comments on commit 249853b

Please sign in to comment.