-
Notifications
You must be signed in to change notification settings - Fork 7
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
Conversation
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.
one nitpick about f interpolator, would appreciate answering other questions too
token: FunctionToken, | ||
args: A, | ||
resource: T | ||
)(using Context): Output[R] |
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.
not exactly correct, missing CallOptions
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.
but we can disregard that for now
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.
yes, you are correct, it's a stub only
"azure-quickstart-acr-geo-replication", | ||
"aws-quickstart-redshift", | ||
"nxos" | ||
) |
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.
what's this?
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.
the packages that are not downloading schemas properly
786e9c3
to
3cb41e0
Compare
96c8268
to
be20140
Compare
be20140
to
f6ede28
Compare
f6ede28
to
0e8b683
Compare
Fixes #283