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

Remove unused Terraform configs #259

Merged
merged 5 commits into from
Sep 26, 2024

Conversation

jeancochrane
Copy link
Contributor

@jeancochrane jeancochrane commented Sep 20, 2024

While testing out ccao-data/actions#33, I noticed that I had forgotten to clean up the Terraform configs that are local to this repo when I factored Batch logic out to the ccao-data/actions repo in #59. Now that the ccao-data/actions repo controls the Terraform config for the build-and-run-model workflow, there's no need to have a config in this repo anymore. This PR implements that missing cleanup step.

@jeancochrane jeancochrane marked this pull request as ready for review September 20, 2024 20:50
Copy link
Member

@wrridgeway wrridgeway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹

@jeancochrane jeancochrane merged commit f52c640 into master Sep 26, 2024
4 checks passed
@jeancochrane jeancochrane deleted the jeancochrane/test-terraform-improvements branch September 26, 2024 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants