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

Re #190 - Fixing grammatical errors and enhancing overall grammar in README.md #195

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

avighnac
Copy link
Contributor

@avighnac avighnac commented Jun 2, 2023

This is an identical pull request to #190 without the lots of changes issue. This should be okay to merge.

This commit addresses grammatical errors found in the project's readme and improves its overall grammar.

Emphasizing the significance of proper grammar cannot be stressed enough when it comes to enhancing the presentation of a project. Today, I came across this GitHub repository in a news article and was intrigued by the concept. Therefore, I decided to explore the repository. To my disappointment, I discovered several grammatical errors in the README.md file. However, the advantage of open source is that I can rectify these errors myself relatively easily if they bother me.

Please inform me if any modifications are necessary.

Copy link
Contributor

@jlopezlira jlopezlira left a comment

Choose a reason for hiding this comment

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

LGTM! But I can't approve it. :(

@di-sukharev 👀

@di-sukharev
Copy link
Owner

@avighnac sorry to be sooo late, was winning top 1 in github hackathon with OpenCommit :)

thank you <3 let me take some time to review it, i should have time on the weekend

@di-sukharev di-sukharev merged commit 0c8bf55 into di-sukharev:dev Jun 19, 2023
1 check passed
@di-sukharev
Copy link
Owner

thank you <3

@avighnac
Copy link
Contributor Author

No worries, and it's amazing that opencommit won in github's hackathon!

I'm just glad that it's finally over haha.

You're welcome!

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.

None yet

3 participants