Skip to content

Grammatically updated the express documentation for better comprehension - #4926

Merged
dougwilson merged 1 commit into
expressjs:masterfrom
REALSTEVEIG:my-contribution
Aug 20, 2022
Merged

Grammatically updated the express documentation for better comprehension#4926
dougwilson merged 1 commit into
expressjs:masterfrom
REALSTEVEIG:my-contribution

Conversation

@REALSTEVEIG

@REALSTEVEIGREALSTEVEIG commented Jun 11, 2022

Copy link
Copy Markdown
Contributor

Adding Js to node just makes it easier for newbies to comprehend the docs.
I suggest the entire docs replaces Nodejs to Node.js.

@ejchengejcheng added docs Documentations issues pr labels Jun 11, 2022

@srkdssrkds left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi,
I think it would be better to change NodeJs to Node.js
as the whole README.md file is following the Node.js style

Comment threadReadme.md Outdated
@dougwilson

Copy link
Copy Markdown
Contributor

Hi @REALSTEVEIG are you planning to update the pr contents with the above agreement?

@REALSTEVEIG

Copy link
Copy Markdown
ContributorAuthor

I don't understand your question.

@dougwilson

Copy link
Copy Markdown
Contributor

Sorry! So above, someone suggested you write it as Node.js instead of NodeJS. You agreed. I was wondering if you were planning to update the pr to use that.

@REALSTEVEIG

Copy link
Copy Markdown
ContributorAuthor

Oh, I will do just that..

@dougwilson
dougwilson merged commit 33e8dc3 into expressjs:masterAug 20, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsDocumentations issuespr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@REALSTEVEIG@dougwilson@srkds@ejcheng