Skip to content

Prefixed namespaces#79

Open
izolight wants to merge 16 commits intolerouxrgd:masterfrom
izolight:prefixed-namespaces
Open

Prefixed namespaces#79
izolight wants to merge 16 commits intolerouxrgd:masterfrom
izolight:prefixed-namespaces

Conversation

@izolight
Copy link
Copy Markdown
Contributor

This MR adds the option to prefix namespaces to generated types.
This is necessary if you have types in multiple namespaces with the same names or having fields with the same name.

@liff
Copy link
Copy Markdown

liff commented Mar 19, 2025

IIRC prost generates modules for the namespaces, so that you get protos::org::example::namespace::Message. Do you think that’d work here as well?

@izolight
Copy link
Copy Markdown
Contributor Author

I am already trying to get the module approach working in a separate branch.
The prefixed was just much easier to get working as my initial goal was to get rid of name conflicts. But seeing how impractical the names get with bigger schemas I might consider withdrawing this once I get the namespace as module working.

@flxo
Copy link
Copy Markdown
Contributor

flxo commented Nov 26, 2025

@izolight thanks for investigation! any updates on the module approach?

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.

3 participants