-
Notifications
You must be signed in to change notification settings - Fork 138
Java: Document @cds.java.ignore
#2097
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
base: main
Are you sure you want to change the base?
Conversation
| This annotation propagates across projections and they also omit the ignored elements. This can be overridden with the following annotation: | ||
|
|
||
| ```cds | ||
| annotate Books:ignored with @cds.java.ignore: false; |
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.
I'm probably lacking modelling knowledge to understand this. How do I set the context where this should be used, for example a service? I guess it's not globally overridden, or?
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.
Not sure if I understand the question.
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.
Where is this annotate statement usually located?
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.
Inside the service where annotation is overridden.
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
|
@renejeglinsky Can we merge this? |
| } | ||
| ``` | ||
|
|
||
| This annotation propagates across projections and they also omit the ignored elements. This can be overridden with the following annotation: |
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.
| This annotation propagates across projections and they also omit the ignored elements. This can be overridden with the following annotation: | |
| This annotation propagates across projections and they also omit the ignored elements. This can be overridden (in a test, for example) with the following annotation: |
Trying to provide an example use case, can ignore it and merge the PR if you'd like. @vmikhailenko
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.
Overriding ignore is very exotic. I can remove whole clause about override to avoid confusion.
No description provided.