Skip to content

Commit

Permalink
No way to NOT select a environment with that key, and we need a way t…
Browse files Browse the repository at this point in the history
…o force it to matrix through everything. Users will just have to update DEPLOY_EXAMPLES temporarly to deploy to a single one
Cameronsplaze committed Oct 31, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
1 parent ec4a2fc commit 6b4c06d
Showing 2 changed files with 2 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/README.md
Original file line number Diff line number Diff line change
@@ -47,7 +47,7 @@ I made this different than `cdk-synth`. For synth, it should run on EVERY config
Technically to do this, all secrets/variables are turned into environment variables in the action. However, even if the container is untrusted, if you don't pass the env-vars to the container in the `./Examples/<container>` yaml file, it won't be able to see them.
**Finally**: If you decide to remove it to save money, you can just remove the one line from `DEPLOY_EXAMPLES`. This lets you keep the environment around with all the variables, in case you want to re-enable it later.
**Finally**: If you decide to remove it to save money, you can just remove the one line from `DEPLOY_EXAMPLES`, then delete the stack. This lets you keep the environment around with all the variables, in case you want to re-deploy it later.
## Forking this Repo
8 changes: 1 addition & 7 deletions .github/workflows/main-pipeline-cdk.yml
Original file line number Diff line number Diff line change
@@ -1,13 +1,7 @@
name: Main Pipeline - CDK Actions

on:
workflow_dispatch:
inputs:
environment:
type: environment
default: all
required: False

workflow_dispatch: {}

pull_request:
types:

0 comments on commit 6b4c06d

Please sign in to comment.