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 resource method generation to codegen #320

Merged
merged 4 commits into from
Dec 4, 2023

Conversation

pawelprazak
Copy link
Contributor

Fixes #283

@pawelprazak pawelprazak added impact/broken Something that is difficult or impossible for some people to use kind/missing We are missing a part of functionality compared to upstream area/codegen Schema to code generator labels Dec 2, 2023
@pawelprazak pawelprazak added this to the 0.2.0 milestone Dec 2, 2023
@pawelprazak pawelprazak requested a review from lbialy December 2, 2023 17:28
Copy link
Collaborator

@lbialy lbialy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one nitpick about f interpolator, would appreciate answering other questions too

token: FunctionToken,
args: A,
resource: T
)(using Context): Output[R]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not exactly correct, missing CallOptions

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but we can disregard that for now

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, you are correct, it's a stub only

"azure-quickstart-acr-geo-replication",
"aws-quickstart-redshift",
"nxos"
)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the packages that are not downloading schemas properly

scripts/Packages.scala Outdated Show resolved Hide resolved
scripts/common.scala Outdated Show resolved Hide resolved
scripts/common.scala Outdated Show resolved Hide resolved
@lbialy lbialy self-requested a review December 4, 2023 08:58
@pawelprazak pawelprazak force-pushed the pprazak/283-add-methods branch from 786e9c3 to 3cb41e0 Compare December 4, 2023 09:01
@pawelprazak pawelprazak requested a review from prolativ as a code owner December 4, 2023 09:01
@pawelprazak pawelprazak force-pushed the pprazak/283-add-methods branch 2 times, most recently from 96c8268 to be20140 Compare December 4, 2023 09:14
@pawelprazak pawelprazak force-pushed the pprazak/283-add-methods branch from be20140 to f6ede28 Compare December 4, 2023 09:16
@pawelprazak pawelprazak force-pushed the pprazak/283-add-methods branch from f6ede28 to 0e8b683 Compare December 4, 2023 09:25
@pawelprazak pawelprazak merged commit cd88836 into develop Dec 4, 2023
3 checks passed
@pawelprazak pawelprazak deleted the pprazak/283-add-methods branch December 4, 2023 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/codegen Schema to code generator impact/broken Something that is difficult or impossible for some people to use kind/missing We are missing a part of functionality compared to upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add missing methods in codegen
2 participants