Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DOC]: Remove TODO statement in docs/source/developer_guide/architecture.md #1877

Closed
2 tasks done
dagardner-nv opened this issue Aug 29, 2024 · 0 comments · Fixed by #1879
Closed
2 tasks done

[DOC]: Remove TODO statement in docs/source/developer_guide/architecture.md #1877

dagardner-nv opened this issue Aug 29, 2024 · 0 comments · Fixed by #1879
Assignees
@dagardner-nv
Labels
doc Improvements or additions to documentation
Milestone
24.10 - Release

Comments

@dagardner-nv
Copy link
Contributor

How would you describe the priority of this documentation request

High

Please provide a link or source to the relevant docs

https://docs.nvidia.com/morpheus/developer_guide/architecture.html

Describe the problems in the documentation

We should probably scan all of the documentation for TODO statements.

* Module Layer
    * TODO: Add details about modules.

(Optional) Propose a correction

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
  • I have searched the open documentation issues and have found no duplicates for this bug report
@dagardner-nv dagardner-nv added the doc Improvements or additions to documentation label Aug 29, 2024
@dagardner-nv dagardner-nv self-assigned this Aug 29, 2024
@dagardner-nv dagardner-nv mentioned this issue Aug 30, 2024
Merged
@morpheus-bot-test morpheus-bot-test bot moved this from Todo to Review - Ready for Review in Morpheus Boards Aug 30, 2024
@mdemoret-nv mdemoret-nv added this to the 24.10 - Release milestone Sep 11, 2024
@github-project-automation github-project-automation bot moved this from Review - Ready for Review to Done in Morpheus Boards Sep 11, 2024
rapids-bot bot pushed a commit that referenced this issue Sep 11, 2024
@dagardner-nv
Remove TODO statements from documentation (#1879)
381097e 
* Remove `TODO` statements from documentation
* Add the word `TODO` to the `reject.txt` file, although due to errata-ai/vale#893 this does not guarantee occurrences will always be flagged.

Closes #1877

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/nv-morpheus/Morpheus/blob/main/docs/source/developer_guide/contributing.md).
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - David Gardner (https://github.com/dagardner-nv)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: #1879
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dagardner-nv dagardner-nv

Labels
doc Improvements or additions to documentation
Projects
Morpheus Boards
Status: Done
Milestone
24.10 - Release
Development

Successfully merging a pull request may close this issue.

Remove TODO statements from documentation dagardner-nv/Morpheus
2 participants
@mdemoret-nv @dagardner-nv