Skip to content

Update Readme.md#1275

Open
Meet2054 wants to merge 1 commit intoWorldBrain:developfrom
Meet2054:patch-1
Open

Update Readme.md#1275
Meet2054 wants to merge 1 commit intoWorldBrain:developfrom
Meet2054:patch-1

Conversation

@Meet2054
Copy link
Copy Markdown

Improving it in formal tone and slightly refined for clarity and grammatical accuracy.

 Improving it in formal tone and slightly refined for clarity and grammatical accuracy.
Copy link
Copy Markdown

@iamtouchskyer iamtouchskyer left a comment

Choose a reason for hiding this comment

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

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!

@iamtouchskyer
Copy link
Copy Markdown

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! 🙏

@iamtouchskyer
Copy link
Copy Markdown

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:
Please visit to our [community forums]
Please visit our [community forums] (remove 'to')

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! 🦞

@iamtouchskyer
Copy link
Copy Markdown

👋 Hi @Meet2054!

As mentioned in the review, there's just one small grammar fix needed:

❌ Current: "Please visit to our [community forums]"
✅ Fix needed: "Please visit our [community forums]" (remove 'to')

Could you please push this one-line fix? Once that's done, this PR will be ready to merge. Thanks for the contribution!

Copy link
Copy Markdown

@iamtouchskyer iamtouchskyer left a comment

Choose a reason for hiding this comment

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

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:

  1. 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.

  2. 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.

@iamtouchskyer
Copy link
Copy Markdown

Hi @Meet2054, thank you for your contribution to improve the README!

As noted in the previous reviews, there are just two small fixes needed:

  1. Line 83: Change "Please visit to our community forums" → "Please visit our community forums" (remove the word 'to')
  2. Line 38: Change "getmemex.com)you" → "getmemex.com) you" (add space after the closing parenthesis)

Once these are fixed, this PR will be ready to merge. The other improvements (US spelling consistency and proper hyphenation) are excellent! 👍

Copy link
Copy Markdown

@iamtouchskyer iamtouchskyer left a comment

Choose a reason for hiding this comment

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

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.

@iamtouchskyer
Copy link
Copy Markdown

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"
Should be: "Please visit our community forums" (remove 'to')

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. 😊

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