Skip to content

fs: throw more errors in JS land #18871

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.

Sign up for GitHub

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

Jump to bottom
Closed
joyeecheung wants to merge 11 commits into from
Closed

fs: throw more errors in JS land #18871

joyeecheung wants to merge 11 commits into from

Conversation

@joyeecheung
Copy link
Member

@joyeecheung joyeecheung commented Feb 19, 2018 •
edited
Loading

Refs: #18106

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

fs

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. fs Issues and PRs related to the fs subsystem / file system. labels Feb 19, 2018
@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 19, 2018

CI: https://ci.nodejs.org/job/node-test-pull-request/13282/

@joyeecheung joyeecheung requested a review from jasnell February 19, 2018 21:13
@joyeecheung joyeecheung added the semver-major PRs that contain breaking changes and should be released in the next major version. label Feb 19, 2018
@BridgeAR BridgeAR self-requested a review February 21, 2018 16:33
@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 21, 2018

Fixed a bug on Windows and error message tests of copyfile. new CI: https://ci.nodejs.org/job/node-test-pull-request/13310/

@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 22, 2018

One error in the last CI but should be unrelated

URL        https://ci.nodejs.org/job/node-test-commit-linux/nodes=debian8-x86/16591/console
Reason
  not ok 1191 parallel/test-net-write-connect-write
    ---
    duration_ms: 0.169
    severity: fail
    stack: |-
    ...

@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 22, 2018

Another linux CI, just to be sure: https://ci.nodejs.org/job/node-test-commit-linux/16612/

This needs two TSC approvals to land, ping @nodejs/tsc

@targos targos self-assigned this Feb 22, 2018
@targos targos removed their assignment Feb 22, 2018
@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 26, 2018

Ping for reviews... @jasnell @BridgeAR @nodejs/tsc

@joyeecheung joyeecheung added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 26, 2018
@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 27, 2018

CI before landing: https://ci.nodejs.org/job/node-test-pull-request/13392/

@joyeecheung
Copy link
Member Author

joyeecheung commented Feb 27, 2018

Landed in 29be1e5...4eb45b8, thanks!

joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw rmdirSync errors in JS
46164ba 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw mkdirSync errors in JS
77b42e3 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw realpathSync.native errors in JS
72d150e 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw readdirSync errors in JS
d2c4f50 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw openSync errors in JS
fea5dda 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: use SyncCall in OpenFileHandle
e8ec898 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw fs.chmodSync errors in JS land
437c756 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw fs.chownSync errors in JS land
8fb5a6c 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw fs.utimesSync errors in JS land
82523d3 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw fs.mkdtempSync errors in JS land
d2dc2a5 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
joyeecheung added a commit that referenced this pull request Feb 27, 2018
@joyeecheung
fs: throw copyFileSync errors in JS
4eb45b8 
PR-URL: #18871
Refs: #18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw rmdirSync errors in JS
2c28999 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw mkdirSync errors in JS
fef4153 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw realpathSync.native errors in JS
ecd9898 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw readdirSync errors in JS
6679c4f 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw openSync errors in JS
e57995f 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: use SyncCall in OpenFileHandle
1e29c61 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw fs.chmodSync errors in JS land
62fe240 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw fs.chownSync errors in JS land
faf1e15 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw fs.utimesSync errors in JS land
05b69a5 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw fs.mkdtempSync errors in JS land
769cc2c 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
@joyeecheung @MayaLekova
fs: throw copyFileSync errors in JS
b896bbc 
PR-URL: nodejs#18871
Refs: nodejs#18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@targos targos targos approved these changes

@BridgeAR BridgeAR BridgeAR approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. fs Issues and PRs related to the fs subsystem / file system. semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@joyeecheung @jasnell @targos @BridgeAR @nodejs-github-bot