-
Notifications
You must be signed in to change notification settings - Fork 701
Review of user manual (only typos and syntax) #1255
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
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.
The commits should probably be squashed into a single commit when we merge, no need for one per file. |
Regarding a squashed merge, I agree. Will have to investigate that first. |
All good, you don't have to do anything for that, there is a "Squash and merge" option we can use on GitHub for it. |
I finished the review now (see Review part2). |
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 am done so far, waiting for comments from reviewer
Co-authored-by: Gerwin Klein <gerwin.klein@proofcraft.systems> Signed-off-by: hexcoder <heiko@hexco.de> Signed-off-by: Gerwin Klein <gerwin.klein@proofcraft.systems>
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.
Have tweaked the two remaining review feedback comments and squashed all commits into one. This is now ready to merge from my side.
I reviewed the user manual up to before section 10.7 (part 1) with regard to typos and consistency. Since I am neither a native speaker nor an expert, please review my changes, thanks.