-
Notifications
You must be signed in to change notification settings - Fork 254
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
[Applications.ComponentBased] Enhance API descriptions #6373
Conversation
Public API ChangedPlease follow the ACR process for the changed API below. Added: 0, Removed: 0, Changed: 1Changed- /// <since_tizen>none</since_tizen
+ /// <since_tizen>6</since_tizen
System.Boolean Tizen.Applications.ComponentBased.Common.ServiceComponent::OnCreate()
|
Public API ChangedPlease follow the ACR process for the changed API below. Added: 0, Removed: 0, Changed: 1Changed- /// <since_tizen>none</since_tizen
+ /// <since_tizen>6</since_tizen
System.Boolean Tizen.Applications.ComponentBased.Common.ServiceComponent::OnCreate()
|
/// <returns>True if a service component is successfully created</returns> | ||
/// <returns> | ||
/// <c>true</c> if the service component is successfully created; otherwise, <c>false</c>. | ||
/// </returns> | ||
public abstract bool OnCreate(); |
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.
Please add a missing since tag.
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.
When I added the since tag, the ACR-Required tag was added.
So I kept it as before.
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 think we can ignore ACR-Requried. Because, the tag was added when ACR was accepted.
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.
Ok I see. I'll prepare the patch.
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.
Thank you.
Public API ChangedPlease follow the ACR process for the changed API below. Added: 0, Removed: 0, Changed: 1Changed- /// <since_tizen>none</since_tizen
+ /// <since_tizen>6</since_tizen
System.Boolean Tizen.Applications.ComponentBased.Common.ServiceComponent::OnCreate()
|
Public API ChangedPlease follow the ACR process for the changed API below. Added: 0, Removed: 0, Changed: 1Changed- /// <since_tizen>none</since_tizen
+ /// <since_tizen>6</since_tizen
System.Boolean Tizen.Applications.ComponentBased.Common.ServiceComponent::OnCreate()
|
Signed-off-by: Changgyu Choi <[email protected]>
Signed-off-by: upple <[email protected]>
Description of Change
Enhance API descriptions
API Changes