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.

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

Fix grammar, typos #788

Merged
merged 2 commits into from
Jan 13, 2024
Merged

Fix grammar, typos #788

merged 2 commits into from
Jan 13, 2024

Conversation

akx
Copy link
Contributor

@akx akx commented Jan 3, 2024

Closes #614 (includes changes from it, plus some reformatting).
Closes #660 (includes changes from it, including a missing userdata_set).

@akx akx changed the title Fix grammar and typos in README Fix some grammar and typos Jan 3, 2024
@akx akx marked this pull request as draft January 3, 2024 08:37
@akx akx changed the title Fix some grammar and typos Fix grammar, typos, license links Jan 3, 2024
@akx akx marked this pull request as ready for review January 3, 2024 08:46
@akx
Copy link
Contributor Author

akx commented Jan 3, 2024

@PierreF The Python test failures are unrelated (see #789).

akx and others added 2 commits January 4, 2024 00:03
Closes eclipse#614

Co-authored-by: alexander-minchin <83529885+alexander-minchin@users.noreply.github.com>
Signed-off-by: Aarni Koskela <akx@iki.fi>
Co-authored-by: jalotra <shivam.jalotra@flipkart.com>
Signed-off-by: Aarni Koskela <akx@iki.fi>
@akx akx changed the title Fix grammar, typos, license links Fix grammar, typos Jan 3, 2024
@PierreF
Copy link
Contributor

PierreF commented Jan 3, 2024

Thank. But I've an issue regarding Eclipse requirement with this PR: since some change are authored by @alexander-minchin which don't seems to have signed the ECA (https://accounts.eclipse.org/user/eca) I'm not sure I'm allowed to merge it.

@alexander-minchin could you complete the ECA signing (https://accounts.eclipse.org/user/eca) with the email you used for the commit (it's the github email: 83529885+alexander-minchin AT users.noreply.github DOT com)

@PierreF
Copy link
Contributor

PierreF commented Jan 13, 2024

Since the change is rather small, I think it's okay to merge it even if first author didn't fully complete the ECA.

@PierreF PierreF merged commit e9f3815 into eclipse:master Jan 13, 2024
10 checks passed
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.

2 participants