Skip to content

Commit

Permalink
fix: add modules script now uses dashes (#193)
Browse files Browse the repository at this point in the history
  • Loading branch information
christian-calabrese authored Dec 24, 2024
1 parent a39432e commit 5477380
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions .github/workflows/push_modules_to_subrepo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ jobs:
needs: [get-modified-modules, get-all-modules]
if: always()
strategy:
fail-fast: false
matrix:
module: ${{ fromJSON(needs.get-all-modules.outputs.modules || needs.get-modified-modules.outputs.modules) }}
steps:
Expand Down
3 changes: 2 additions & 1 deletion infra/scripts/add-module.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ if [ -z "$MODULE_NAME" ]; then
fi

DX_PREFIX="dx"
SUBREPO_NAME="terraform-$PROVIDER-${DX_PREFIX}_${MODULE_NAME}"
SUBREPO_NAME="terraform-$PROVIDER-${DX_PREFIX}-${MODULE_NAME}"
SUBREPO_NAME=${SUBREPO_NAME//_/\-}
MODULE_DIR="infra/modules/$MODULE_NAME"

# Check if the module directory already exists
Expand Down

0 comments on commit 5477380

Please sign in to comment.