Skip to content

fixture: add the process of throwing exceptions - #2480

Merged
1 commit merged into
masterfrom
unknown repository
Sep 1, 2019
Merged

1 commit merged into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Aug 30, 2019

Copy link
Copy Markdown
  1. When there's something wrong in 'fs.readdir' in either 'server.js' or 'build.js', this is an async callback, so we should throw exceptions immediately to stop the futher process to let users know what mistakes they may have made.

  2. Change the Log info for 'add' and 'change' for file watching.

1) When there's something wrong in 'fs.readdir' in either 'server.js' or 'build.js',
this is an async callback, so we should throw exceptions immediately to
stop the futher process to let users know what mistakes they may have
made.
2) Change the Log info for 'add' and 'change' for file watching.
@ghost
ghost requested review from alexandrtovmach and removed request for alexandrtovmach August 30, 2019 05:41
@ghost
ghost merged commit 9958008 into nodejs:master Sep 1, 2019
@ghost
ghost deleted the NewThrowEx branch September 1, 2019 02:25
This pull request was closed.
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.

1 participant