doc: add documentation for blob.bytes() method

[jmsb02]

Description:
This pull request adds documentation for the blob.bytes() method in the buffer module. The method was introduced in Node.js v20.16.0, but it was not documented. This PR aims to provide a clear description and examples for the blob.bytes() method.

Motivation:
The blob.bytes() method allows users to obtain the bytes of a Blob object as a Buffer.
This method was introduced in a recent release but is currently undocumented.

Fixes: #54105

Documentation:

blob.bytes()

The blob.bytes() method returns the bytes of the Blob object as a Buffer.

const blob = new Blob(['hello']);
console.log(blob.bytes()); // Outputs: <Buffer 68 65 6c 6c 6f>

Additional Notes:
I included the text.txt file incorrectly while testing git push for the first time. I'd appreciate it if you could ignore it. I'll be more careful in the future!

[jakecastelli]

Thanks for the contribution! The added doc is incorrect and this PR can be improved as the following:

• blob.bytes() returns aPromise<Uint8Array> instead of Buffer
• remove the unrelated file change as other reviewer pointed (remove test.txt)
• please follow the commit message guide line where you can find it here in this case I would suggest commit message to be doc: add documentation for blob.bytes() method

Please let me know if you need any assistance!

[jmsb02]

Thank you for the feedback. I have made the following changes as per the suggestions:

1. Removed the unrelated test.txt file.
2. Updated the blob.bytes() method documentation to reflect the correct return type of Promise<Uint8Array>.

blob.bytes()

The blob.bytes() method returns the bytes of the Blob object as a Promise<Uint8Array>.

const blob = new Blob(['hello']);
blob.bytes().then((bytes) => {
  console.log(bytes); // Outputs: Uint8Array(5) [ 104, 101, 108, 108, 111 ]
});

Please let me know if there are any additional changes required. Thanks!

[jakecastelli]

Do you mind squashing your 2 commits into 1 commit?

[jmsb02]

I have squashed the 2 commits into 1 commit as requested. Please let me know if there are any additional changes required. Thank you :)

[jmsb02]

I have added the version information as you requested. Thank you for pointing that out. If there are any further changes needed, please let me know. I apologize for any inconvenience caused.

[jakecastelli]

No worries about the mistakes, you have been doing well! There is a pipeline failure due to linting, do you mind looking into it? 👀 Should be an easy fix!

[jmsb02]

I've just made the corrections you mentioned. As this is my first time, it's been challenging, but thanks to your help, I'm managing to get through it step by step. Thank you very much!

[jakecastelli]

Your last commit (198bfa6) was not correct, the one before that (6574a6a) is the correct linting fix. Could you drop the last commit (198bfa6)? As you can see your last commit had many irrelevant changes.

[jmsb02]

I have reset the branch to the correct commit (6574a6a) and force-pushed the changes. The incorrect commit (198bfa6) has been removed. Thank you for your guidance!

Blockers have been addressed

[jmsb02]

I have moved the blob.bytes() entry after blob.arrayBuffer so that it is in alphabetical order. Please let me know if there are any additional changes required. Thank you!

[targos]

Suggested change

[targos]

Suggested change

[targos]

Suggested change

	added:
	- v20.16.0
	added:
	- v22.3.0
	- v20.16.0

[jmsb02]

I have reflected your comments. Thank you!

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/54114
✔  Done loading data for nodejs/node/pull/54114
----------------------------------- PR info ------------------------------------
Title      doc: add documentation for blob.bytes() method  (#54114)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     jmsb02:add-blob-bytes-doc -> nodejs:main
Labels     author ready, commit-queue-squash
Commits    5
 - doc: add documentation for blob.bytes() method
 - doc: add documentation for blob.bytes() method
 - doc: add documentation for blob.bytes() method
 - doc: add documentation for blob.bytes() method
 - doc: add documentation for blob.bytes() method
Committers 1
 - jaexxin <jmsb02@naver.com>
PR-URL: https://github.com/nodejs/node/pull/54114
Fixes: https://github.com/nodejs/node/issues/54105
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/54114
Fixes: https://github.com/nodejs/node/issues/54105
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 30 Jul 2024 06:37:26 GMT
   ✔  Approvals: 1
   ✔  - Daeyeon Jeong (@daeyeon): https://github.com/nodejs/node/pull/54114#pullrequestreview-2217050112
   ✘  This PR needs to wait 65 more hours to land (or 0 hours if there is one more approval)
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/10228146599

[jakecastelli]

Landed in 76d10a1

[jakecastelli]

Hi @jmsb02 I manually landed the PR for you as your first commit message body contains duplicated information. Congrats to become a contributor 🥳