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

doc: add ESM examples in http.md #47763

Closed
wants to merge 3 commits into from
Closed

doc: add ESM examples in http.md #47763

wants to merge 3 commits into from

Conversation

btea
Copy link
Contributor

@btea btea commented Apr 28, 2023

http module example code update.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/http
  • @nodejs/net

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

tniessen commented May 2, 2023

cc @nodejs/documentation @nodejs/modules

@tniessen tniessen 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.

@btea btea changed the title doc: example code add mjs code doc: add ESM examples in http.md Aug 8, 2023
@btea
Copy link
Contributor Author

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,

doc/api/http.md Outdated Show resolved Hide resolved
doc/api/http.md Outdated Show resolved Hide resolved
doc/api/http.md Outdated Show resolved Hide resolved
lpinca pushed a commit that referenced this pull request Aug 10, 2023
PR-URL: #47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@lpinca
Copy link
Member

lpinca commented Aug 10, 2023

Landed in 48c613e.

@lpinca lpinca closed this Aug 10, 2023
@btea btea 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
PR-URL: nodejs#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Aug 15, 2023
PR-URL: #47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@UlisesGascon UlisesGascon mentioned this pull request Aug 15, 2023
RafaelGSS pushed a commit to RafaelGSS/node that referenced this pull request Aug 15, 2023
PR-URL: nodejs#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
rluvaton pushed a commit to rluvaton/node that referenced this pull request Aug 15, 2023
PR-URL: nodejs#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Aug 17, 2023
PR-URL: #47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Nov 27, 2023
PR-URL: #47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
PR-URL: nodejs/node#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
PR-URL: nodejs/node#47763
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem. review wanted PRs that need reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants