-
Notifications
You must be signed in to change notification settings - Fork 459
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
feat(lib): Terraform Attributes (token system replacement) #706
Conversation
Cool - thanks for pushing on the topic 🎉 Gonna dig into this a bit deeper over the next few days |
this._operations.forEach(operation => { | ||
reference = operation(reference); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like this!
As discussed with @jsteinich, @DanielMSchmidt we won't pursue this proposal. We're going to address the issues we have within the existing token system. @jsteinich thank you again for the effort you put into this PR and the input during the entire process. It sparked important discussions and helped all of us to build a common understanding of the problem domain. |
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Closes #525
Fixes #518
Fixes #374
Fixes #434
Fixes #424
Partially implements #25
Fixes #172
Fixes #663
Fixes #679
Copied from issue comment
Current problems
Other desired expansions
Solution preface
Proposed solution
TerraformAttribute
classTerraformStringAttribute
,TerraformNumberAttribute
,TerraformBooleanAttribute
,TerraformDynamicAttribute
,TerraformListAttribute
,TerraformSetAttribute
,TerraformMapAttribute
,TerraformObjectAttribute
TerraformAttribute
sTerraformAttribute
(or subtype) being created if not already using oneUpdates
putX
methodsScope of work
Initial work
Immediate follow-up
Longer term follow-up