Skip to content

rn-117: minor copy edit #740

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 1 commit into from
Dec 1, 2024
Merged

rn-117: minor copy edit #740

merged 1 commit into from
Dec 1, 2024

Conversation

stepnem
Copy link
Contributor

@stepnem stepnem commented Dec 1, 2024

No description provided.

@chriscool
Copy link
Collaborator

LGTM, but I'd like @mjaix to take a look if possible.

@mjaix
Copy link
Contributor

mjaix commented Dec 1, 2024

LGTM, too, contains everything corrected from a missing comma (where I had asked myself whether to tolerate it or not) to more severe stuff. It is simply great to have you on board.
After merging, I would like to spend about half an hour (at most) to go over the complete edition once more, if time allows.

@mjaix mjaix merged commit ec5acb1 into git:master Dec 1, 2024
@mjaix
Copy link
Contributor

mjaix commented Dec 1, 2024 via email

@chriscool
Copy link
Collaborator

@mjaix and @stepnem, it looks great to me. Thanks both!

@stepnem stepnem deleted the rn117copyedit branch March 3, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants