Skip to content

Conversation

@btea
Copy link
Contributor

@bteabtea commented Apr 28, 2023

http module example code update.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/http
  • @nodejs/net

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem. labels Apr 28, 2023
@bteabtea changed the title docs: example code add mjs codedoc: example code add mjs codeApr 28, 2023
@tniessen
Copy link
Member

cc @nodejs/documentation @nodejs/modules

@tniessentniessen added the review wanted PRs that need reviews. label May 2, 2023
@lpinca
Copy link
Member

lpinca commented Aug 8, 2023

@btea can you please amend the first commit message and make it start with an imperative verb? Something like "doc: add ESM examples in http.md" should work.

@bteabtea changed the title doc: example code add mjs codedoc: add ESM examples in http.mdAug 8, 2023
@btea
Copy link
ContributorAuthor

btea commented Aug 8, 2023

@btea can you please amend the first commit message and make it start with an imperative verb? Something like "doc: add ESM examples in http.md" should work.

Thanks for the tip, I have updated the message,

lpinca pushed a commit that referenced this pull request Aug 10, 2023
@lpinca
Copy link
Member

Landed in 48c613e.

@lpincalpinca closed this Aug 10, 2023
@bteabtea deleted the docs/http-mjs branch August 10, 2023 12:58
martenrichter pushed a commit to martenrichter/node that referenced this pull request Aug 13, 2023
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
RafaelGSS pushed a commit that referenced this pull request Aug 15, 2023
@UlisesGasconUlisesGascon mentioned this pull request Aug 15, 2023
RafaelGSS pushed a commit to RafaelGSS/node that referenced this pull request Aug 15, 2023
rluvaton pushed a commit to rluvaton/node that referenced this pull request Aug 15, 2023
RafaelGSS pushed a commit that referenced this pull request Aug 17, 2023
targos pushed a commit that referenced this pull request Nov 27, 2023
@targostargos mentioned this pull request Nov 28, 2023
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.httpIssues or PRs related to the http subsystem.review wantedPRs that need reviews.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@btea@nodejs-github-bot@tniessen@lpinca@guybedford