-
Notifications
You must be signed in to change notification settings - Fork 10
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
[COMPLETED]DOCUMENTATION Update #19
Conversation
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.
Please move the changes to README.md
and delete documentation.md
. That is where it's meant to be.
Add a point about the requirement of Amrita WiFi/VPN to access the tool in usage as a separate step.
alright!! |
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.
So far looks good @RD-Tarun
Consider adding Amrita Logo
, Go
logo, and possibly some relevant shield icons.
done, please check it out @Ashrockzzz2003 @Abhinav-ark |
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.
Thanks for linking my portfolio instead of GitHub! : )
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.
LGTM
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.
LGTM.
/bounty 50 @RD-Tarun |
#3 - PR Issue Link
@Ashrockzzz2003 I've created the documentation for the CLI tool in OSS Format as well. Please review the changes and let me know if any further modifications are needed.
Thank You!!