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

Replace null_resource with terraform_data #40

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jansiwy
Copy link
Member

@jansiwy jansiwy commented Jan 7, 2025

null_resource is deprecated, as there is built-in terraform_data instead which can cover the use cases of null_resources (apart from other use cases):

https://developer.hashicorp.com/terraform/language/resources/terraform-data

Depends on:

  • Discontinnue support for older Terraform versions

@jansiwy jansiwy requested a review from a team January 7, 2025 16:24
@jansiwy jansiwy self-assigned this Jan 7, 2025
Copy link

deepsource-io bot commented Jan 7, 2025

Here's the code health analysis summary for commits 096c6d9..a41cd9d. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗
DeepSource Terraform LogoTerraform✅ Success
🎯 1 occurence resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@jtsaito
Copy link
Contributor

jtsaito commented Jan 7, 2025

Hm, it seems CI is running on Terraform version 1.1.9 but the new feature requires 1.9:https://github.com/hashicorp/terraform/pull/35171/files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants