Conversation
|
My biggest concern with this is the added maintainability burden (external URLs, duplicated content), but otherwise I'm okay with keeping it if deemed useful for customers. @lloadholt I've added you as reviewer because most of this is documentation. You can find the internal-only, built documentation here. |
Duplication is a concern, but is something we long ago decided was a lesser evil in the Connectors. Trying to consolidate docs for all Connectors would be harder to implement and maintain that some limited duplication. |
…mmunity/rticonnextdds-connector-rust into feature/user-guide-initial
@lloadholt - Most of this documentation is adapted from the JavaScript connector, so I don't think it requires a deep review. |
lloadholt
left a comment
There was a problem hiding this comment.
Commented through 'getting-started.md'.
lloadholt
left a comment
There was a problem hiding this comment.
Commented remaining md files (index.md through result.md)
This comment was marked as resolved.
This comment was marked as resolved.
|
@alexcamposruiz No further comments on the content. Is it ok for me to edit these files (and the Connector for JavaScript manual) directly to apply RTI style? Primarily, I'd like to change the text format for the following words to match the format used in other RTI user manuals:
|
@lloadholt Sure. Can you approve this PR and create new ones with your edits? |
|
I am merging this PR and deleting the branch. In the future, prefer making the changes in your own clone, to avoid issues with rulesets and avoid polluting the |
Summary
Details
Where Previous Content Moved
Areas Needing More Work
JS Parity Gaps
These are potential improvements, but may not be necessary for now:
Other