Skip to content

export type ctx()#6

Open
massemanet wants to merge 1 commit into
tsloughter:masterfrom
massemanet:master
Open

export type ctx()#6
massemanet wants to merge 1 commit into
tsloughter:masterfrom
massemanet:master

Conversation

@massemanet

@massemanet massemanet commented Jun 22, 2023

Copy link
Copy Markdown

For some reason, files generated from .proto with gpb refers to both ctx:t() and ctx:ctx() as types.
This PR exports ctx:ctx() so that this actually works.

@tsloughter

Copy link
Copy Markdown
Owner

Thanks but the fix should be to have the generated protos only use ctx:t().

@massemanet

massemanet commented Jun 22, 2023 via email

Copy link
Copy Markdown
Author

@tsloughter

Copy link
Copy Markdown
Owner

Actually, this was fixed in 0.9.0 tsloughter/grpcbox_plugin@f88f6ae

Are you on the latest version (0.9.0)?

@massemanet

massemanet commented Jun 23, 2023 via email

Copy link
Copy Markdown
Author

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