-
Notifications
You must be signed in to change notification settings - Fork 22
Refactor documentation #499
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
base: main
Are you sure you want to change the base?
Conversation
Hello @JoeZiminski, I have added a few comments based on my observations regarding the user experience. Please have a look, and feel free to ignore anything that may not be necessary. Thank you! |
Hey @JoeZiminski I think this PR also contains the contents of #492. |
15c0889
to
7bd8e28
Compare
Thanks Both! Ah yes I did branch off as you mention @cs7-shrey, once #492 is merged I will rebase this back on main and drop those commits! |
This PR reworks the documentation to be more accessible for first-time users.