-
Notifications
You must be signed in to change notification settings - Fork 4
Docs: Modified Installation_Guide of docs #1133
base: develop
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1133 +/- ##
==========================================
Coverage ? 86.93%
==========================================
Files ? 85
Lines ? 4057
Branches ? 237
==========================================
Hits ? 3527
Misses ? 458
Partials ? 72 Continue to review full report at Codecov.
|
docs/Installation_Guide.md
Outdated
### A one-time setup: | ||
|
||
- Install python | ||
```bash |
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 dedent this
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 check
I'd be also in favour of somehow integrating https://github.com/anitab-org/vms/pull/1081/files |
Do i need to integrate full windows doc or only the part you highlighted? |
Let's include a link like "Hey, if you're from windows, go here" or something similar |
Co-authored-by: Sanket Dasgupta <[email protected]>
Lol I didn't touch the test cases ! |
docs/Installation_Guide.md
Outdated
10. [Change Directory Permissions](https://github.com/systers/vms/blob/master/docs/Installation%20Guide.md#change-directory-permissions) | ||
11. [Run Development Server](https://github.com/systers/vms/blob/master/docs/Installation%20Guide.md#run-development-server) | ||
12. [Run Unit Tests](https://github.com/systers/vms/blob/master/docs/Installation%20Guide.md#run-unit-tests) | ||
## For Linux/MacOS User |
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.
Half of the commands won't work for Mac OS.
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.
Owh I didn't know , I will remove this! Is there a dedicated doc for MacOs?
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.
I think I saw one on the wiki or somewhere
Any more suggestion or should I squash the commits finally @SanketDG ? |
Description
Modified outdated version of docs/Installation_Guide.md with newer version of auth_docs/Installation_Setup.md
Fixes #962
Type of Change:
Checklist: