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

schema: Conclude module_variable_optional_attrs language experiment (v1.8) #328

Closed
radeksimko opened this issue Mar 1, 2024 · 1 comment · Fixed by #337
Closed

schema: Conclude module_variable_optional_attrs language experiment (v1.8) #328

radeksimko opened this issue Mar 1, 2024 · 1 comment · Fixed by #337
Assignees
Labels
enhancement New feature or request

Comments

@radeksimko
Copy link
Member

See hashicorp/terraform#34752

schema.Keyword{
Keyword: "module_variable_optional_attrs",
Name: "feature",
},

I would also recommend reviewing experiments in general, to check they actually align with core.

@radeksimko radeksimko added the enhancement New feature or request label Mar 1, 2024
@radeksimko radeksimko changed the title schema: Conclude `` language experiment (v1.8) schema: Conclude module_variable_optional_attrs language experiment (v1.8) Mar 1, 2024
@apparentlymart
Copy link

Given that the whole point of language experiments is that they be lightweight to add, remove, and conclude, I would suggest that this repository probably shouldn't try to mirror the list of them, and instead tools should consider the experiments argument as just a static list of arbitrary identifiers, leaving Terraform Core to check whether they are valid.

Given how short-lived some experiments tend to be, it seems like overkill to try to do autocomplete and semantic checking for individual experiment keywords. Having to update the schema for every single lifecycle change of every single experiment seems like it would either cause this set to always be constantly outdated (the current situation) or cause teams to waste time trying to carefully chase changes to a very-marginally-used part of the language that is never valid in stable releases anyway.

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
None yet
Development

Successfully merging a pull request may close this issue.

4 participants