📝 Add docstrings to develop#41
Conversation
Docstrings generation was requested by @vinniefalco. * #40 (comment) The following files were modified: * `src/corosio/src/detail/epoll/sockets.hpp` * `src/corosio/src/detail/iocp/sockets.hpp` * `src/corosio/src/socket.cpp` * `test/unit/socket.cpp`
|
Important Review skippedThis PR was authored by the user configured for CodeRabbit reviews. By default, CodeRabbit skips reviewing PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
An automated preview of the documentation is available at https://41.corosio.prtest3.cppalliance.org/index.html If more commits are pushed to the pull request, the docs will rebuild at the same URL. 2026-01-21 00:22:52 UTC |
|
@CodeRabbit Nothing in "detail", "impl" directories, or "detail" namespaces is public. They dont need added docstrings. |
|
Tip For best results, initiate chat on the files or code changes.
In this PR, that means the following files should not have had docstrings added:
I'll remember this for future docstring generation to focus only on the public API. ✏️ Learnings added
|
Docstrings generation was requested by @vinniefalco.
The following files were modified:
src/corosio/src/detail/epoll/sockets.hppsrc/corosio/src/detail/iocp/sockets.hppsrc/corosio/src/socket.cpptest/unit/socket.cppThese files were kept as they were
include/boost/corosio/io_stream.hppinclude/boost/corosio/socket.hppℹ️ Note