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.

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

docs: updates docstrings for the CLI commands #350

Merged
merged 2 commits into from
Oct 10, 2024
Merged

docs: updates docstrings for the CLI commands #350

merged 2 commits into from
Oct 10, 2024

Conversation

pallabpain
Copy link
Member

@pallabpain pallabpain commented Sep 8, 2024

Description

Adds extensive documentation for the CLI commands.

@pallabpain pallabpain self-assigned this Sep 8, 2024
@pallabpain pallabpain added the 📚 documentation Improvements or additions to documentation label Sep 8, 2024
Copy link

github-actions bot commented Sep 8, 2024

🤖 Pull Request Artifacts (#11276141364) 🎉

@pallabpain pallabpain force-pushed the docs/cli branch 15 times, most recently from d77e20e to c98719c Compare September 14, 2024 06:06
@pallabpain pallabpain force-pushed the docs/cli branch 2 times, most recently from 65dd7a7 to 42ea95b Compare September 15, 2024 17:41
@pallabpain pallabpain marked this pull request as ready for review September 15, 2024 17:49
@pallabpain pallabpain requested review from a team as code owners September 15, 2024 17:49
@pallabpain pallabpain changed the title 📚 docs: updates docstrings for the CLI commands docs: updates docstrings for the CLI commands Sep 15, 2024
Copy link
Member Author

@pallabpain pallabpain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TOD List

  1. Update README and copy the same context in sphinx index.
  2. Use the new rapyuta.io logo.

@pallabpain pallabpain force-pushed the docs/cli branch 2 times, most recently from 2b8642c to 5e1264d Compare September 16, 2024 12:15
@pallabpain pallabpain force-pushed the docs/cli branch 10 times, most recently from 093e06f to cf13e17 Compare September 17, 2024 11:21
@pallabpain pallabpain force-pushed the docs/cli branch 2 times, most recently from 8fb6afb to 00effe6 Compare October 10, 2024 12:56
CONTRIBUTING.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
riocli/apply/__init__.py Outdated Show resolved Hide resolved
riocli/deployment/util.py Show resolved Hide resolved
riocli/deployment/wait.py Outdated Show resolved Hide resolved
riocli/device/delete.py Outdated Show resolved Hide resolved
riocli/device/metric.py Outdated Show resolved Hide resolved
riocli/device/onboard.py Outdated Show resolved Hide resolved
riocli/device/tools/rapyuta_logs.py Show resolved Hide resolved
The CLI has lacked extensive documentation for most part
and that leaves its users confused about its usage.

This commit attempts to solve that problem by adding some
elaborate docstrings to the commands with examples such
that the users can figure out things by reading it.
Copy link
Member Author

@pallabpain pallabpain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ankitrgadiya I have addressed the comments. Please take a look.

@ankitrgadiya ankitrgadiya merged commit 75720d6 into devel Oct 10, 2024
7 checks passed
@pallabpain pallabpain deleted the docs/cli branch October 14, 2024 04:57
@rr-github-ci-user
Copy link
Collaborator

🎉 This PR is included in version 9.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📚 documentation Improvements or additions to documentation released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants