Skip to content

Commit 0669712

Browse files
deokjinkimdanielleadams
authored andcommitted
doc: use console.error for error case in fs, https, net and process
console.error is more suitable than console.log for error case. PR-URL: #45606 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it>
1 parent fa0a2d8 commit 0669712

File tree

4 files changed

+15
-15
lines changed

4 files changed

+15
-15
lines changed

doc/api/fs.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -937,15 +937,15 @@ try {
937937
await copyFile('source.txt', 'destination.txt');
938938
console.log('source.txt was copied to destination.txt');
939939
} catch {
940-
console.log('The file could not be copied');
940+
console.error('The file could not be copied');
941941
}
942942

943943
// By using COPYFILE_EXCL, the operation will fail if destination.txt exists.
944944
try {
945945
await copyFile('source.txt', 'destination.txt', constants.COPYFILE_EXCL);
946946
console.log('source.txt was copied to destination.txt');
947947
} catch {
948-
console.log('The file could not be copied');
948+
console.error('The file could not be copied');
949949
}
950950
```
951951

doc/api/https.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ let https;
2424
try {
2525
https = require('node:https');
2626
} catch (err) {
27-
console.log('https support is disabled!');
27+
console.error('https support is disabled!');
2828
}
2929
```
3030

@@ -42,7 +42,7 @@ let https;
4242
try {
4343
https = await import('node:https');
4444
} catch (err) {
45-
console.log('https support is disabled!');
45+
console.error('https support is disabled!');
4646
}
4747
```
4848

doc/api/net.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,7 @@ after a certain amount of time:
407407
```js
408408
server.on('error', (e) => {
409409
if (e.code === 'EADDRINUSE') {
410-
console.log('Address in use, retrying...');
410+
console.error('Address in use, retrying...');
411411
setTimeout(() => {
412412
server.close();
413413
server.listen(PORT, HOST);

doc/api/process.md

+10-10
Original file line numberDiff line numberDiff line change
@@ -3190,7 +3190,7 @@ if (process.getegid && process.setegid) {
31903190
process.setegid(501);
31913191
console.log(`New gid: ${process.getegid()}`);
31923192
} catch (err) {
3193-
console.log(`Failed to set gid: ${err}`);
3193+
console.error(`Failed to set gid: ${err}`);
31943194
}
31953195
}
31963196
```
@@ -3204,7 +3204,7 @@ if (process.getegid && process.setegid) {
32043204
process.setegid(501);
32053205
console.log(`New gid: ${process.getegid()}`);
32063206
} catch (err) {
3207-
console.log(`Failed to set gid: ${err}`);
3207+
console.error(`Failed to set gid: ${err}`);
32083208
}
32093209
}
32103210
```
@@ -3235,7 +3235,7 @@ if (process.geteuid && process.seteuid) {
32353235
process.seteuid(501);
32363236
console.log(`New uid: ${process.geteuid()}`);
32373237
} catch (err) {
3238-
console.log(`Failed to set uid: ${err}`);
3238+
console.error(`Failed to set uid: ${err}`);
32393239
}
32403240
}
32413241
```
@@ -3249,7 +3249,7 @@ if (process.geteuid && process.seteuid) {
32493249
process.seteuid(501);
32503250
console.log(`New uid: ${process.geteuid()}`);
32513251
} catch (err) {
3252-
console.log(`Failed to set uid: ${err}`);
3252+
console.error(`Failed to set uid: ${err}`);
32533253
}
32543254
}
32553255
```
@@ -3280,7 +3280,7 @@ if (process.getgid && process.setgid) {
32803280
process.setgid(501);
32813281
console.log(`New gid: ${process.getgid()}`);
32823282
} catch (err) {
3283-
console.log(`Failed to set gid: ${err}`);
3283+
console.error(`Failed to set gid: ${err}`);
32843284
}
32853285
}
32863286
```
@@ -3294,7 +3294,7 @@ if (process.getgid && process.setgid) {
32943294
process.setgid(501);
32953295
console.log(`New gid: ${process.getgid()}`);
32963296
} catch (err) {
3297-
console.log(`Failed to set gid: ${err}`);
3297+
console.error(`Failed to set gid: ${err}`);
32983298
}
32993299
}
33003300
```
@@ -3325,7 +3325,7 @@ if (process.getgroups && process.setgroups) {
33253325
process.setgroups([501]);
33263326
console.log(process.getgroups()); // new groups
33273327
} catch (err) {
3328-
console.log(`Failed to set groups: ${err}`);
3328+
console.error(`Failed to set groups: ${err}`);
33293329
}
33303330
}
33313331
```
@@ -3338,7 +3338,7 @@ if (process.getgroups && process.setgroups) {
33383338
process.setgroups([501]);
33393339
console.log(process.getgroups()); // new groups
33403340
} catch (err) {
3341-
console.log(`Failed to set groups: ${err}`);
3341+
console.error(`Failed to set groups: ${err}`);
33423342
}
33433343
}
33443344
```
@@ -3369,7 +3369,7 @@ if (process.getuid && process.setuid) {
33693369
process.setuid(501);
33703370
console.log(`New uid: ${process.getuid()}`);
33713371
} catch (err) {
3372-
console.log(`Failed to set uid: ${err}`);
3372+
console.error(`Failed to set uid: ${err}`);
33733373
}
33743374
}
33753375
```
@@ -3383,7 +3383,7 @@ if (process.getuid && process.setuid) {
33833383
process.setuid(501);
33843384
console.log(`New uid: ${process.getuid()}`);
33853385
} catch (err) {
3386-
console.log(`Failed to set uid: ${err}`);
3386+
console.error(`Failed to set uid: ${err}`);
33873387
}
33883388
}
33893389
```

0 commit comments

Comments
 (0)