-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat(deprecated): generate language specific deprecation markup #156
feat(deprecated): generate language specific deprecation markup #156
Conversation
…deprecated text in comments If @deprecated is present in a comment, a deprecation marker will be generated. @deprecated is expected to be at the start of a line, any text following it on the same line is added as the ‘message’ text where supported This applies to records/attributes/constants, interfaces/methods/constants, enums/values, flags/values Supports C++, Java and ObjC generators
f8b6960
to
64cf608
Compare
Wow, this is nice! Thank you so much!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great PR!
We need to add this to the documentation.
But that can happen once it's merged.
Could be nice if this would also support setting the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apart from that it's looking great, thank you so much!
If no other comments are made, I will soon merge this PR, and we will handle eventual ideas and additions, like docs, as extra issues. |
Thanks a lot! |
Adds support for special handling of the @deprecated annotation in IDL comments. When encountered, the generator will output language specific deprecation markup. This change is a port of the existing deprecation support in the cross-language-cpp repository. Ported "feat(deprecated): generate language specific deprecation markup" Original author: @eakoli Original PR: cross-language-cpp/djinni-generator#156
Adds support for special handling of the @deprecated annotation in IDL comments. When encountered, the generator will output language specific deprecation markup. This change is a port of the existing deprecation support in the cross-language-cpp repository. Ported "feat(deprecated): generate language specific deprecation markup" Original author: @eakoli Original PR: cross-language-cpp/djinni-generator#156
If @deprecated is present in a comment, a deprecation marker will be generated.
@deprecated is expected to be at the start of a line, any text following it on the same line is added as the ‘message’ text where supported
This applies to records/attributes/constants, interfaces/methods/constants, enums/values, flags/values
Supports C++, Java and ObjC generators