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

Add helper for future vpc resources #1234

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Add helper for future vpc resources #1234

merged 1 commit into from
Jun 20, 2024

Conversation

annakhm
Copy link
Collaborator

@annakhm annakhm commented Jun 19, 2024

This helper will be used in autogenerated vpc resources that don't have explicit parent path in schema

@ksamoray
Copy link
Collaborator

I'd put this in utl_file_template.yaml which will place this here after make api-wrapper.
Capitalize the func name, so it will be accessible (and while so, also the linter won't complain about..)

This helper will be used in autogenerated vpc resources that
don't have explicit parent path in schema

Signed-off-by: Anna Khmelnitsky <[email protected]>
@annakhm
Copy link
Collaborator Author

annakhm commented Jun 19, 2024

I'd put this in utl_file_template.yaml which will place this here after make api-wrapper. Capitalize the func name, so it will be accessible (and while so, also the linter won't complain about..)

That's a good idea, but I prefer to wait with it until we have a single home place for the wrapper generator and templates

@annakhm annakhm merged commit 72eb537 into master Jun 20, 2024
3 checks passed
@annakhm annakhm deleted the vpc-helper branch August 7, 2024 20:36
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