Automatically pick up all files in ./eslint-local-rules

[fregante]

Once the rules start to pile up, it's easier to keep them in separate files. Currently I have this in index.js:

const fs = require("node:fs");
const path = require("node:path");

const ruleFiles = fs
  .readdirSync(__dirname)
  .filter((file) => !file.endsWith("test.js") && !file.endsWith("index.js"));

const rules = Object.fromEntries(
  ruleFiles.map((file) => [
    path.basename(file, ".js"),
    require("./" + file),
  ])
);

module.exports = rules;

It would be good to add something like it on the condition that:

• eslint-local-rules.js isn't found
• eslint-local-rules/index.js isn't found
• eslint-local-rules/*.js files exist

This follows #22 in that it reduces the amount of boilerplate code that one needs to copy-paste. For people who want custom loading logic, they can keep using the index file.

[cletusw]

That sounds reasonable to me! Feel free to send a PR and I'll try not to take as long this time 😬

[fregante]

I'm not sure what I meant by this, the configs in #23 already do this.

I think the only part missing is the automatic exclusion of .test.js files after getting the list of .js files:

eslint-plugin-local-rules/index.js

Line 8 in 7c47aa8

var rules = requireUp('eslint-local-rules', DEFAULT_EXTENSIONS, process.cwd());