Skip to content

Conversation

@TheUncertaintim
Copy link
Contributor

Description

This PR fixed a few typos in the code snippets provided in the Learn/Asynchronous Work section.

While working on the PR, a hyperlink to the pull request template was found to be broken in the docs. This PR fixed that also.

Validation

Typo

The function named sendMail() is now mentioned correctly in the comment (instead of sendEmail())

image

Broken link

For the broken link, I can now jump to the correct file (PULL_REQUEST_TEMPLATE.md) instead of pull_request_template.md while browsing in VS Code Markdown Preview.

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

@TheUncertaintim TheUncertaintim requested review from a team as code owners July 2, 2025 22:37
@vercel
Copy link

vercel bot commented Jul 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview Jul 2, 2025 10:39pm

@MattIPv4
Copy link
Member

MattIPv4 commented Jul 2, 2025

👀 I realise these are both small changes, but could you split the contributing docs change into a separate PR, please, so that your PRs contain logical changesets?

@TheUncertaintim
Copy link
Contributor Author

Sure.
#7940

#7941

@avivkeller avivkeller closed this Jul 3, 2025
@avivkeller
Copy link
Member

Closing in favor

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.

3 participants