-
-
Notifications
You must be signed in to change notification settings - Fork 5k
no-new-object to allow argument #11810
Copy link
Copy link
Closed
Labels
archived due to ageThis issue has been archived; please open a new issue for any further discussionThis issue has been archived; please open a new issue for any further discussionauto closedThe bot closed this issueThe bot closed this issueenhancementThis change enhances an existing feature of ESLintThis change enhances an existing feature of ESLintevaluatingThe team will evaluate this issue to decide whether it meets the criteria for inclusionThe team will evaluate this issue to decide whether it meets the criteria for inclusionruleRelates to ESLint's core rulesRelates to ESLint's core rules
Metadata
Metadata
Assignees
Labels
archived due to ageThis issue has been archived; please open a new issue for any further discussionThis issue has been archived; please open a new issue for any further discussionauto closedThe bot closed this issueThe bot closed this issueenhancementThis change enhances an existing feature of ESLintThis change enhances an existing feature of ESLintevaluatingThe team will evaluate this issue to decide whether it meets the criteria for inclusionThe team will evaluate this issue to decide whether it meets the criteria for inclusionruleRelates to ESLint's core rulesRelates to ESLint's core rules
Type
Fields
Give feedbackNo fields configured for issues without a type.
What rule do you want to change?
no-new-objectDoes this change cause the rule to produce more or fewer warnings?
Fewer warnings.
How will the change be implemented? (New option, new default behavior, etc.)?
New default behavior.
Please provide some example code that this change will affect:
What does the rule currently do for this code?
Reports an error.
What will the rule do after it's changed?
Not report anything.
Are you willing to submit a pull request to implement this change?
Yes.
As per the above example, there are uses for wrapping some items within a
new Object()(or within the equivalent,Object(), but some prefer the style of requiringnewin such cases, as perunicorn/new-for-builtins).If it is thought that people will like to make this optional, I can modify to something like
allowWithArguments.