-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
Fix network documentation coverage #72992
Fix network documentation coverage #72992
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.
Nice work! There are some compliance/spelling checks that need fixing.
f070d8d
to
fc7af9a
Compare
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.
Nice! :-) Approved for IEEE 802.15.4 part.
fc7af9a
to
b775d0f
Compare
|
Added missing doxygen comments. Also hide internal symbols from generated documentation. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
b775d0f
to
563ca5e
Compare
Fix the documentation in ptp_time.h to improve the documentation coverage. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Fix doxygen documentation in sntp.h header file. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Some of the internal macros can be hidden from generated documentation. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
No need to generate documentation for internal data. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Fix the socket_offload.h header file documentation. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
No need to generate public documentation for internal data. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Do not add internal symbols to generated documentation. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
No need to generate documentation for internal data. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Add missing doxygen documentation to socketcan.h header file. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
This will improve documentation coverage. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
The @brief was in wrong place. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
The trickle.h was missing couple of doxygen comments. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
The virtual API was not properly documented. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Opcodes were not documented. Also removed init function from public documentation as it is not needed there. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Add missing doxygen comments and improve documentation coverage. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Add missing doxygen comments for wifi_mgmt.h header file. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
No need to generate documentation for internal data in wifi_nm.h Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
The wifi_utils.h was missing couple of doxygen comments. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
The zperp.h header file was missing doxygen comments. Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
306c8d5
to
2b5fb70
Compare
|
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, thanks!
This should improve network documentation coverage percentage.
So I added missing doxygen comments when applicable or hide internal documentation which does not need to be documented.
There are lot of commits, but most of them are very small. Each header file changed is placed in its own commit for easier revert / fix if needed.