8000 Review of user manual (only typos and syntax) by hexcoder- · Pull Request #1255 · seL4/seL4 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Conversation

hexcoder-
Copy link
Contributor

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.

Copy link
Member
@lsf37 lsf37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for that, this is great!

@Indanz, @bbrcknl does this conflict with other PR's you have open on updating the manual?

@lsf37 lsf37 requested a review from Indanz May 26, 2024 23:18
@lsf37
Copy link
Member
lsf37 commented May 26, 2024

The commits should probably be squashed into a single commit when we merge, no need for one per file.

@lsf37 lsf37 added the docs Manual and other documentation label May 26, 2024
@hexcoder-
Copy link
Contributor Author

Regarding a squashed merge, I agree. Will have to investigate that first.

@lsf37
Copy link
Member
lsf37 commented May 27, 2024

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.

@hexcoder-
Copy link
Contributor Author

I finished the review now (see Review part2).

Copy link
Contributor Author
@hexcoder- hexcoder- left a 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>
Copy link
Member
@lsf37 lsf37 left a 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.

@lsf37 lsf37 self-assigned this Jun 17, 2024
@lsf37 lsf37 merged commit 15fa2cf into seL4:master Jun 17, 2024
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Manual and other documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0