Skip to content

Conversation

@fliiiix
Copy link
Contributor

@fliiiix fliiiix commented Jun 24, 2025

This prevents the include of the type defined in the pyi, otherwise this leads to error message like this:
error: Name XYZ already defined (possibly by an import) [no-redef]

@github-actions github-actions bot added c++ codegen Involving generating code from schema python labels Jun 24, 2025
@fliiiix fliiiix force-pushed the bugfix/double-include-own-type branch from 3725316 to cce6539 Compare June 25, 2025 06:15
@fliiiix fliiiix force-pushed the bugfix/double-include-own-type branch from cce6539 to 07788af Compare July 8, 2025 11:34
@fliiiix
Copy link
Contributor Author

fliiiix commented Jul 8, 2025

@akb825 any opinions on this change?

@akb825
Copy link
Contributor

akb825 commented Jul 8, 2025

Looks good to me.

@fliiiix
Copy link
Contributor Author

fliiiix commented Jul 14, 2025

@aardappel or @dbaileychess can this be merged?

@fliiiix fliiiix force-pushed the bugfix/double-include-own-type branch from 07788af to 11efd6b Compare July 15, 2025 07:29
This prevents the include of the type defined in the pyi,
otherwise this leads to error message like this:
error: Name XYZ already defined (possibly by an import)  [no-redef]
@fliiiix fliiiix force-pushed the bugfix/double-include-own-type branch from 11efd6b to 62e4da6 Compare July 15, 2025 11:08
@aardappel aardappel merged commit c7b9dc8 into google:master Jul 15, 2025
51 checks passed
@aardappel
Copy link
Collaborator

Thanks!

@fliiiix fliiiix deleted the bugfix/double-include-own-type branch July 16, 2025 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ codegen Involving generating code from schema python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants