-
Notifications
You must be signed in to change notification settings - Fork 66
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: clarify the docs #45
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.
Thanks for your contribution @steveoh. There's a few details that will require some change before being merged but I think there are some solid improvements here. Great stuff! 🥰
Co-Authored-By: Ben Schwarz <ben@calibreapp.com>
Co-Authored-By: Ben Schwarz <ben@calibreapp.com>
Co-Authored-By: Ben Schwarz <ben@calibreapp.com>
Co-Authored-By: Ben Schwarz <ben@calibreapp.com>
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 made a couple of small suggestions for readability or formatting, but otherwise this is ready to go.
Co-Authored-By: Ben Schwarz <ben@calibreapp.com>
Great contribution, thanks so much @steveoh |
I got a warning that my configuration was out dated. When I read the docs about how to update, I was a little confused and it took a few more more runs to get it right. This pr aims to fix that and a few other things. I think it reads better and I hope you do too.