Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates defaults and adds better support for using CodeDeploy with ECS/Fargate #60

Merged
merged 16 commits into from
Sep 28, 2023

Conversation

JaredDarling
Copy link
Contributor

No description provided.

@JaredDarling
Copy link
Contributor Author

@jnonino it look like a secret for INFRACOST_API_KEY must be added to GitHub for the workflow action to run successfully.

@jnonino jnonino added the enhancement New feature or request label Sep 22, 2023
@jnonino
Copy link
Member

jnonino commented Sep 22, 2023

Hi @JaredDarling, thanks for your contributions, I removed the Infracost stage. If you rebase that step will not be there, and pipeline will succeed. Same with your other PR.

@jnonino jnonino enabled auto-merge (squash) September 22, 2023 09:01
jnonino
jnonino previously approved these changes Sep 22, 2023
auto-merge was automatically disabled September 27, 2023 16:43

Head branch was pushed to by a user without write access

@JaredDarling
Copy link
Contributor Author

@jnonino Updated branch and all checks pass now. Ready for final approval.

@jnonino jnonino merged commit 97fdb60 into cn-terraform:main Sep 28, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

3 participants