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

feat: sifuncs, resource_values, paths, and input #5291

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

Conversation

sprutton1
Copy link
Contributor

@sprutton1 sprutton1 commented Jan 19, 2025

Add siFuncs and lays out the props like:

  • all props go in the domain
  • create- and write-only props in the domain get input sockets and related attrFuncs
  • read-only props in the domain get output sockets that they are inputs to
  • all read-only props go in resource_value
  • they get added as inputs in the related domain prop

This means things like VpcId that come back from a resource get added to /root/resource_value/VpcId, then /root/domain/VpcId then as an output socket with the value.

Other domain props can be set normally or via the related input socket.

This includes setting the paths, but the path for objects and maps might be... wrong. For now. Deal with it.

image

Copy link

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

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.

1 participant