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

Updating Dev.md for better readability #381

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

SamarthBagga
Copy link

In this pull request, I've updated the dev.md file to enhance its readability. The changes involve improved formatting and better wording in some places. I have also added colons before each command/code block.

@SamarthBagga
Copy link
Author

SamarthBagga commented Jan 14, 2024

Hi, @meganindya could you please review this pr?

@SamarthBagga
Copy link
Author

Hi, @walterbender could you please review this pr?


This is a _**TypeScript**_ project that uses _**React**_. You'll just need
_[**Node.js**](https://nodejs.org/en) v16_ and _**npm**_ installed on your development machine.
_[**Node.js**](https://nodejs.org/en) v16_ or higher and _**npm**_ installed on your development machine.
Copy link
Member

Choose a reason for hiding this comment

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

We should confirm this assertion.

Copy link
Author

Choose a reason for hiding this comment

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

@walterbender, I have tested it on v18 and v20 and it works fine on both.

docs/DEV.md Outdated
@@ -258,10 +264,3 @@ _keyboard shortcuts_, etc.
_**Visual Studio Code**_ (_**VSCode**_) is currently the most-popular code editor for reasons like
being _lightweight_, _cleaner_, large marketplace of _extensions_, integrated _source control_
features, _debugger_, _remote explorer_ support, _regular expression_ based find/replace, etc.

Copy link
Member

Choose a reason for hiding this comment

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

Any reason you deleted this section?

Copy link
Author

Choose a reason for hiding this comment

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

@walterbender, I accidentally removed it, i have now reverted the change.

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