Closed bmish closed 3 years ago
not sure why having the option. I think it's fine to just remove it, as users cannot publish eslint core?
I assume it was originally intended for users who wanted to submit a pull request for a new rule to ESLint core. Maybe some people still want to use it for that? I don't feel strongly but I did want to make sure it was not the default behavior.
It's used to adjust the rule test import:
var rule = require("../../../lib/rules/<%= ruleId %>"),
<% if (target === "eslint") { %>
RuleTester = require("../../../lib/testers/rule-tester");
<% } else { %>
RuleTester = require("eslint").RuleTester;
<% } %>
highly likely to be. cc @nzakas
Yes, we wanted the generator to help people both write their own plugins as well as contribute to the core. Im fine with moving plugin first.
Since it's more common for consumers to use this for generating ESLint plugin rules than ESLint core rules. We want the more common usage to be the default (first).
In this CLI prompt: