Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Allow for the PluginError constructor to be called without new. #77

Closed
wants to merge 3 commits into from
Closed

Allow for the PluginError constructor to be called without new. #77

wants to merge 3 commits into from

Conversation

dead-claudia
Copy link

This also falls in line with the builtin Error types and most of the other builtins as well.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4e124b9 on impinball:master into fb4df4b on gulpjs:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d528546 on impinball:master into fb4df4b on gulpjs:master.

@sindresorhus
Copy link
Contributor

👎 It should be explicit that it's a new instance.

@yocontra
Copy link
Member

yocontra commented Jan 7, 2015

This will add a new item to the error stack and push other stuff off. You can manipulate the stack to remove PluginError from it

@phated
Copy link
Member

phated commented Jun 6, 2016

Closing, as work shouldn't be done in this module anymore.

@phated phated closed this Jun 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants