Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DOC] RDoc for process.c #8353

Merged
merged 8 commits into from Sep 2, 2023
Merged

[DOC] RDoc for process.c #8353

merged 8 commits into from Sep 2, 2023

Conversation

BurdetteLamar
Copy link
Member

No description provided.

@BurdetteLamar BurdetteLamar added the Documentation Improvements to documentation. label Sep 1, 2023
peterzhu2118
peterzhu2118 reviewed Sep 1, 2023
View reviewed changes
process.c Outdated Show resolved Hide resolved
process.c Outdated Show resolved Hide resolved
process.c Outdated Show resolved Hide resolved
process.c Outdated Show resolved Hide resolved
process.c Show resolved Hide resolved
process.c Outdated Show resolved Hide resolved
@BurdetteLamar
Copy link
Member Author

Some of these methods could probably profit by having links to the outboard system docs, and maybe to Wikipedia articles; could be in next PR?

@peterzhu2118
Copy link
Member

@BurdetteLamar I don't see most of these comments as being addressed. Did you miss a commit?

@BurdetteLamar
Copy link
Member Author

@BurdetteLamar I don't see most of these comments as being addressed. Did you miss a commit?

I pushed the one revision, then tried to commit your suggestions; missed the message that the diff was outdated. Will try again.

BurdetteLamar and others added 5 commits September 1, 2023 18:52
@BurdetteLamar @peterzhu2118
Update process.c
d6be11c 
Co-authored-by: Peter Zhu <peter@peterzhu.ca>
@BurdetteLamar @peterzhu2118
Update process.c
3d14646 
Co-authored-by: Peter Zhu <peter@peterzhu.ca>
@BurdetteLamar @peterzhu2118
Update process.c
b915b4f 
Co-authored-by: Peter Zhu <peter@peterzhu.ca>
@BurdetteLamar @peterzhu2118
Update process.c
1810748 
Co-authored-by: Peter Zhu <peter@peterzhu.ca>
@BurdetteLamar @peterzhu2118
Update process.c
935b7e1 
Co-authored-by: Peter Zhu <peter@peterzhu.ca>
peterzhu2118
peterzhu2118 reviewed Sep 2, 2023
View reviewed changes
process.c Outdated Show resolved Hide resolved
@BurdetteLamar @peterzhu2118
Update process.c
764a25e 
Co-authored-by: Peter Zhu <peter@peterzhu.ca>
@peterzhu2118 peterzhu2118 merged commit cfcb4a4 into ruby:master Sep 2, 2023
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peterzhu2118 peterzhu2118 peterzhu2118 approved these changes

Assignees
No one assigned
Labels
Documentation Improvements to documentation.
Milestone
No milestone
2 participants
@BurdetteLamar @peterzhu2118