-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Fix: Add script for Windows (non-WSL) users + update readme #185
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.
Just a couple of minor addition suggestions to the README. Good spot on the additional spelling mistakes (thanks 😅)
Add Mac OS as well Co-authored-by: Sayvai <[email protected]>
Add Mac OS as well Co-authored-by: Sayvai <[email protected]>
Forgot about Mac OS part - thanks for the suggestions! 😄 |
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.
Changes look good! Thank you 🙌
Ha, no worries! Now, only @srbhr have the privileges to merge into the base |
Pull Request Title
Add script for Windows (non-WSL) users + update readme for webapp
Related Issue
#153
Description
The existing setup and instructions are not compatible for Windows (non-WSL) users.
Type
Proposed Changes
Screenshots / Code Snippets (if applicable)
N/A
How to Test
Checklist
Additional Information
Also corrected a few typos in readme file