-
Notifications
You must be signed in to change notification settings - Fork 18
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
Clean up adapter documentation #208
Conversation
After moving adapters to `core.adapters`, we forgot to also modify the documentation. This fixes the documentation, plus the core APIs are now alphabetically ordered which is easier to scan through.
7600ebf
to
8afe97e
Compare
lewis/adapters/__init__.py
Outdated
@@ -18,7 +18,7 @@ | |||
# ********************************************************************* | |||
|
|||
""" | |||
The core Adapter API is located in :mod:`lewis.core.adapters`. This module on the contains | |||
The core Adapter API is located in :mod:`lewis.core.adapters`. This package on the contains |
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.
👍
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.
Just the one comment about dashes. Not sure if you agree?
I also went ahead and changed "module" to "package" in adapters/__init__.py
.
|
||
.. toctree:: | ||
:maxdepth: 2 | ||
|
||
adapters/adapters | ||
adapters/epics | ||
adapters/modbus | ||
adapters/stream |
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.
The titles this generates (based on the rst files in docs/api/adapters/
) are:
The EPICS-Adapter
The Modbus Adapter
The Stream-Adapter
The dashes are inconsistent. I think no dash makes more sense?
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.
Yes, seems better without dashes at all, while we're at it.
After moving adapters to
core.adapters
, we forgot to also modify the documentation. This fixes the documentation, plus the core APIs are now alphabetically ordered which is easier to scan through.