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

Update README.md #4394

Merged
merged 1 commit into from
Jul 9, 2024
Merged

Update README.md #4394

merged 1 commit into from
Jul 9, 2024

Conversation

amalikmuhd
Copy link
Contributor

@amalikmuhd amalikmuhd commented Jun 6, 2024

Summary

This PR fixes a few typos and improves the readability of the README file. These changes help ensure clarity and consistency in the documentation.

Changes

  • Fixed the phrasing: "code for which is also on open source" to "code for which is also open source"
  • Clarified the example for refactoring: "Refactoring the codebase, eg to replace mobx with redux or something" to "Refactoring the codebase, e.g., to replace MobX with Redux or something"
  • Updated "day to day" to "day-to-day"
  • Enhanced the phrasing for third-party integration: "third-party can be as seamless as first-party" to "third-party integration can be as seamless as first-party"

Motivation

Ensuring the documentation is free of typos and written clearly helps new developers understand and contribute to the project more effectively.

Checklist

  • The README has been updated to correct typos and improve clarity.
  • All changes have been reviewed and approved.
  • No functionality of the codebase has been altered.

Fix typos and improve readability in README

- Corrected "Github" to "GitHub"
- Fixed "code for which is also on open source" to "code for which is also open source"
- Clarified "Refactoring the codebase, eg to replace mobx with redux or something" to "Refactoring the codebase, e.g., to replace MobX with Redux or something"
- Updated "day to day" to "day-to-day"
- Enhanced phrasing for third-party integration: "third-party can be as seamless as first-party" to "third-party integration can be as seamless as first-party"
@amalikmuhd amalikmuhd marked this pull request as draft June 6, 2024 14:47
@amalikmuhd amalikmuhd marked this pull request as ready for review June 6, 2024 14:49
@haileyok
Copy link
Contributor

haileyok commented Jul 9, 2024

Thanks

@haileyok haileyok merged commit 4360087 into bluesky-social:main Jul 9, 2024
estrattonbailey added a commit that referenced this pull request Jul 9, 2024
…cial-proof

* origin/main: (126 commits)
  Update README.md (#4394)
  tweak top padding external (#4755)
  change `contentVisibility` to `contain` (#4752)
  Fix RTL text rendering for display names (#4747)
  Reduce the size of the inner logo in the QR code (#4746)
  Fix misplaced '@' in RTL post meta. (#4531)
  Remove broken and void back button (#4744)
  Ensure `/start` navigates to `/starter-pack` when clicking a link internally (#4745)
  Add missing `to` in StarterPackScreen.tsx string (#4743)
  Video compression in composer (#4638)
  fix slop (#4739)
  Update stats
  Run intl:extract
  Update Japanese translation (#4665)
  Update Korean localization (#4646)
  Update Chinese Localization (#4695)
  Update catalan (#4702)
  Update Indonesian translation (#4706)
  Tweak checkmark size
  Show feedback for Follow button in interstitials (#4738)
  ...
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