Skip to content
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 @javax.annotation.Generated to generated flatbuffer Java types #4986

Merged
merged 2 commits into from
Oct 8, 2018

Conversation

riklund
Copy link
Contributor

@riklund riklund commented Oct 8, 2018

This is needed to allow tooling to correctly infer that these types are generated.

@aardappel
Copy link
Collaborator

Thanks! Can you run sh generate_code.sh to make sure generated code is updated.

@riklund
Copy link
Contributor Author

riklund commented Oct 8, 2018

Done. Attached updated goldens.

@aardappel
Copy link
Collaborator

Thanks!

@aardappel aardappel merged commit 233976c into google:master Oct 8, 2018
@riklund riklund deleted the bfr2 branch October 8, 2018 22:30
@stewartmiles
Copy link
Contributor

As discussed on another thread, we'll need a way to use this without injection of these annotations as they add a dependency upon jsr250 on Android which some folks - like our project - do not want. @riklund any chance you could make this conditional ?

aardappel added a commit that referenced this pull request Nov 2, 2018
#4986 missed part of the
implementation.

Change-Id: Ie41771eb018a550c289c77ebe9ef5c50d1ad6751
@aardappel
Copy link
Collaborator

@stewartmiles @riklund
Added missing code that makes it conditional here: 2159191

shivendra14 pushed a commit to shivendra14/flatbuffers that referenced this pull request Nov 13, 2018
…oogle#4986)

* Add @javax.annotation.Generated to generated flatbuffer Java types.

* Updating test goldens.
shivendra14 pushed a commit to shivendra14/flatbuffers that referenced this pull request Nov 13, 2018
google#4986 missed part of the
implementation.

Change-Id: Ie41771eb018a550c289c77ebe9ef5c50d1ad6751
zchee pushed a commit to zchee/flatbuffers that referenced this pull request Feb 14, 2019
…oogle#4986)

* Add @javax.annotation.Generated to generated flatbuffer Java types.

* Updating test goldens.
zchee pushed a commit to zchee/flatbuffers that referenced this pull request Feb 14, 2019
google#4986 missed part of the
implementation.

Change-Id: Ie41771eb018a550c289c77ebe9ef5c50d1ad6751
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