-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve documentation #101
Conversation
for more information, see https://pre-commit.ci
Do we really need doc8 to annoy us if the line is too long? I thought we discussed this at some point and anyway one would use alt+z in vscode or something for writing doc |
To check the compiled version you can have a look here 😊 |
Agreed, if possible I would remove all linting of docfiles. |
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 agree with merging
Pull Request Test Coverage Report for Build 6694151483
💛 - Coveralls |
I started writing some documentation a couple of weeks ago but I can't find the time to finish it the next time.
However, I think the things that are added are already helpful for beginners. For this reason, I would suggest merging it at this stage and then we can pick it up from this point the next time someone has time to write something.