Skip to content

docs: update options in filehandle.appendFile() #56972

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

Closed
wants to merge 2 commits into from
Rate limit · GitHub

Access has been restricted

You have triggered a rate limit.

Please wait a few minutes before you try again;
in some cases this may take up to an hour.

Conversation

y-hsgw
Copy link
Contributor

@y-hsgw y-hsgw commented Feb 9, 2025

I updated the documentation for the filehandle.appendFile() options.
And I also added related tests.

@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label Feb 9, 2025
@lpinca
Copy link
Member

lpinca commented Feb 10, 2025

Please replace docs: with doc: in the second commit.

@y-hsgw y-hsgw force-pushed the docs/fs-file-handle-append-file branch from d40ef79 to 47fb2d6 Compare February 10, 2025 14:09
Copy link

codecov bot commented Feb 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.34%. Comparing base (b181535) to head (20fdc99).
Report is 122 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #56972      +/-   ##
==========================================
+ Coverage   89.13%   90.34%   +1.21%     
==========================================
  Files         665      629      -36     
  Lines      193165   184403    -8762     
  Branches    37191    36036    -1155     
==========================================
- Hits       172181   166603    -5578     
+ Misses      13729    10918    -2811     
+ Partials     7255     6882     -373     

see 132 files with indirect coverage changes

@lpinca
Copy link
Member

lpinca commented Feb 10, 2025

Can you please rebase and foce push to keep only two commits? Thank you.

@y-hsgw y-hsgw force-pushed the docs/fs-file-handle-append-file branch from 78c5b07 to 42595e2 Compare February 11, 2025 02:12
@lpinca lpinca added request-ci Add this label to start a Jenkins CI on a PR. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. labels Feb 17, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Feb 17, 2025
@nodejs-github-bot
Copy link
Collaborator

@y-hsgw y-hsgw force-pushed the docs/fs-file-handle-append-file branch from 42595e2 to c8fa484 Compare February 23, 2025 23:56
@y-hsgw y-hsgw force-pushed the docs/fs-file-handle-append-file branch from c8fa484 to 20fdc99 Compare February 24, 2025 01:29
@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Feb 24, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Feb 24, 2025
@nodejs-github-bot
Copy link
Collaborator

@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 24, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 24, 2025
@nodejs-github-bot
Copy link
Collaborator

Landed in 8f4d306...22ac597

nodejs-github-bot pushed a commit that referenced this pull request Feb 24, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
nodejs-github-bot pushed a commit that referenced this pull request Feb 24, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@y-hsgw y-hsgw deleted the docs/fs-file-handle-append-file branch February 24, 2025 23:09
targos pushed a commit that referenced this pull request Feb 25, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Feb 25, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Feb 25, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Feb 25, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 1, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 1, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 1, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 1, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Apr 2, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Apr 2, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Apr 3, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Apr 3, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 16, 2025
PR-URL: #56972
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Apr 16, 2025
PR-URL: #56972
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
commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rate limit · GitHub

Access has been restricted

You have triggered a rate limit.

Please wait a few minutes before you try again;
in some cases this may take up to an hour.

3 participants