Skip to content

Conversation

maxburke
Copy link
Contributor

@maxburke maxburke commented Mar 5, 2023

This PR adds a command line option to skip generation of the type prefixed/suffixed functions in Python code, for example for a Flatbuffer type MyType, this will skip generating "GetRootAsMyType", "MyTypeStart", "MyTypeEnd", "MyTypeAddSomeField", ...

@github-actions github-actions bot added c++ codegen Involving generating code from schema python labels Mar 5, 2023
@github-actions github-actions bot added the grpc label Mar 11, 2023
dbaileychess
dbaileychess previously approved these changes Mar 15, 2023
@dbaileychess
Copy link
Collaborator

@maxburke looks like it just needs to be synced at head.

@maxburke
Copy link
Contributor Author

@dbaileychess What's left for this PR in order to be merged?

@dbaileychess dbaileychess enabled auto-merge (squash) April 6, 2023 01:47
@dbaileychess dbaileychess disabled auto-merge April 6, 2023 01:48
@dbaileychess
Copy link
Collaborator

@maxburke Sorry for the delay. For some reason I don't have a UpdateBranch button like I typically do. Can you remerge?

@maxburke
Copy link
Contributor Author

@dbaileychess done

@dbaileychess dbaileychess merged commit 56ecc1f into google:master Apr 26, 2023
@dbaileychess
Copy link
Collaborator

Sorry, my day job has been keeping me busy.

jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
…e#7857)

* optionally generate type prefixes and suffixes for python code

* fix codegen error when qualified name is empty

* generated code updated
jochenparm pushed a commit to jochenparm/flatbuffers that referenced this pull request Oct 29, 2024
…e#7857)

* optionally generate type prefixes and suffixes for python code

* fix codegen error when qualified name is empty

* generated code updated
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 grpc python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants