Conversation
Improving it in formal tone and slightly refined for clarity and grammatical accuracy.
iamtouchskyer
left a comment
There was a problem hiding this comment.
Thanks for improving the README! Most of these changes look good, but there's a small grammar issue in the last change:
❌ "Please visit to our [community forums]"
✅ "Please visit our [community forums]" (remove 'to')
or alternatively:
✅ "Head over to our [community forums]"
Could you please fix this grammar error? The other changes (spacing fix, US spelling consistency, and proper hyphenation of 'end-to-end') are great improvements!
|
Hi @Meet2054! 👋 Thanks again for your contribution to improve the README. The changes you made are really helpful for clarity and consistency. There's just one small fix needed as mentioned in the review: changing "Please visit to our" to "Please visit our" (removing the word "to"). Would you be able to make this quick update? It's just a one-word change and then this PR should be ready to merge! Thanks for your patience! 🙏 |
|
Hi @Meet2054! 👋 Thanks for your contribution to improve the README! As per the code review from @iamtouchskyer, there's just one small grammar fix needed: Line that needs fixing: Once you address this, the PR should be ready to merge! Your other improvements (spacing, US spelling consistency, and proper hyphenation) look great. Thanks for contributing to Memex! 🦞 |
|
👋 Hi @Meet2054! As mentioned in the review, there's just one small grammar fix needed: ❌ Current: "Please visit to our [community forums]" Could you please push this one-line fix? Once that's done, this PR will be ready to merge. Thanks for the contribution! |
iamtouchskyer
left a comment
There was a problem hiding this comment.
Thanks for the improvements to the README! I appreciate the effort to make the language more formal and clear. However, I noticed a couple of small issues:
-
Grammar issue: "Please visit to our community forums" should be either "Please visit our community forums" or "Head over to our community forums" - the combination "visit to" is incorrect.
-
Missing space: "getmemex.com)you" should be "getmemex.com) you" (missing space after the closing parenthesis).
The other changes look great:
- "organise" → "organize" (consistent American spelling)
- "end2end" → "end-to-end" (proper hyphenation)
Could you please fix these two small issues? After that, this PR will be ready to merge.
|
Hi @Meet2054, thank you for your contribution to improve the README! As noted in the previous reviews, there are just two small fixes needed:
Once these are fixed, this PR will be ready to merge. The other improvements (US spelling consistency and proper hyphenation) are excellent! 👍 |
iamtouchskyer
left a comment
There was a problem hiding this comment.
Thanks for improving the README! Most of your changes look great - fixing the spacing, standardizing to American English spelling, and improving the hyphenation.
However, there's a grammatical issue in the last change:
'Please visit to our community forums' should be 'Please visit our community forums' (remove 'to').
Could you fix this small issue? After that, this PR will be ready to merge.
|
Hi @Meet2054, thank you for your README improvements! 🎉 I noticed there have been multiple reviews pointing out the same grammar issue, but it hasn't been addressed yet. Could you please make this quick fix? Current: "Please visit to our community forums" This is just a one-word removal. Once this is fixed, I'll be happy to merge your contribution immediately! Thanks for your patience and for helping improve the project documentation. 😊 |
Improving it in formal tone and slightly refined for clarity and grammatical accuracy.