Skip to content

Commit

Permalink
fix(workflows): streamline environment input handling in CI configura…
Browse files Browse the repository at this point in the history
…tions

- Removed the required environment input from `sub-ci-integration-tests-gcp.yml`.
- Updated comments in `sub-deploy-integration-tests-gcp.yml` to clarify the strategy for creating images in dev and prod environments based on the main branch.
  • Loading branch information
gustavovalverde committed Dec 26, 2024
1 parent ac02227 commit 2d187bd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/sub-ci-integration-tests-gcp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,6 @@ name: Integration Tests on GCP
on:
workflow_call:
inputs:
environment:
required: true
type: string
network:
default: "Mainnet"
type: string
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/sub-deploy-integration-tests-gcp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -388,6 +388,8 @@ jobs:
permissions:
contents: 'read'
id-token: 'write'
# We want to create images for both dev and prod environments,
# but for prod we only want to create images from the main branch
strategy:
matrix:
environment: ${{ github.ref_name == 'main' && fromJSON('["dev", "prod"]') || fromJSON('["dev"]') }}
Expand Down

0 comments on commit 2d187bd

Please sign in to comment.