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 installation instructions in RMB Peer, Direct README.md #2679

Merged
merged 6 commits into from
May 14, 2024

Conversation

Mahmoud-Emad
Copy link
Contributor

@Mahmoud-Emad Mahmoud-Emad commented May 12, 2024

Description

Update installation instructions in RMB Peer, Direct README.md

Changes

  • Added step-by-step instructions for installing dependencies
  • Clarified system requirements

Related Issues

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstrings
  • Screenshots/Video attached (needed for UI changes)

- Added step-by-step instructions for installing dependencies
- Clarified system requirements
@Mahmoud-Emad Mahmoud-Emad changed the title docs: Updated the rmb docs. Update installation instructions in RMB Peer, Direct README.md May 13, 2024
@Mahmoud-Emad Mahmoud-Emad marked this pull request as ready for review May 13, 2024 08:05
packages/rmb_direct_client/DEVELOPERS.README.md Outdated Show resolved Hide resolved
packages/rmb_direct_client/DEVELOPERS.README.md Outdated Show resolved Hide resolved
packages/rmb_direct_client/DEVELOPERS.README.md Outdated Show resolved Hide resolved
packages/rmb_direct_client/DEVELOPERS.README.md Outdated Show resolved Hide resolved
packages/rmb_peer_client/README.md Outdated Show resolved Hide resolved
- Update DEVELOPERS.README.md in rmb_direct_client to refine the guide and correct typos.
- Update README.md in rmb_peer_client to specify that it's a TypeScript library.
@MohamedElmdary
Copy link
Member

why did we change typescript version? also using 4.7 not 5.x? @Mahmoud-Emad

@Mahmoud-Emad
Copy link
Contributor Author

why did we change typescript version? also using 4.7 not 5.x? @Mahmoud-Emad

There was an error when tried to build the types, and @AhmedHanafy725 suggested that we put this version

@maayarosama maayarosama self-requested a review May 14, 2024 08:51
@maayarosama maayarosama dismissed a stale review May 14, 2024 08:51

Approved by mistake

@Mahmoud-Emad
Copy link
Contributor Author

@xmonader any changes??

@amiraabouhadid amiraabouhadid merged commit 3bb13b1 into development May 14, 2024
2 checks passed
@amiraabouhadid amiraabouhadid deleted the development_rmb_docs branch May 14, 2024 12:57
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.

6 participants