Skip to content

Commit

Permalink
fix(build): repair pdk yarn invocations (#154)
Browse files Browse the repository at this point in the history
In `build.sh`, the invocation to PDK had extra sets of tokens that were
unneeded; I have removed those.

In both `deployAll.sh` and `deployDev.sh`, `pdk` referred to a projen
tasks (`workspace`, `run`) that didn't exist; I have reverted the PDK
invocation to use `yarn` instead.
  • Loading branch information
Daniels-Aaron-bah authored Dec 18, 2024
1 parent 8c41d6a commit 120bf36
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion scripts/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,4 @@ set -e

pdk install --frozen-lockfile
pdk build
pdk workspaces run eslint
pdk eslint
3 changes: 2 additions & 1 deletion scripts/deployAll.sh
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
#!/bin/bash

set -e

SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )

${SCRIPT_DIR}/build.sh

pdk workspace @aws/threat-composer-infra run cdk deploy
yarn workspace @aws/threat-composer-infra run cdk deploy
2 changes: 1 addition & 1 deletion scripts/deployDev.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,4 @@ SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )

${SCRIPT_DIR}/build.sh

pdk workspace @aws/threat-composer-infra run cdk deploy Dev/ThreatComposerAppStack
yarn workspace @aws/threat-composer-infra run cdk deploy Dev/ThreatComposerAppStack

0 comments on commit 120bf36

Please sign in to comment.