-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adds description for all methods within server and client c files wit… #2
Adds description for all methods within server and client c files wit… #2
Conversation
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.
In general, we are on the right path. Good job. We just need to expand our descriptions and comments to be a bit more thorough.
@NihadF7 @marcoraddatz This is approved, we can merge it. I would also advise that we include a new ticket during code implementation which will include additional creation of the documentation for data types,function and use cases in a .md format, as mentioned in resolved comments. This can be done during code implementation as a acceptance criteria. |
I agree with @ssalkic proposal. PR merged. Closing as done. |
Added descriptions to methods for lwm2mserver and lwm2mclient c files