Problem running benchmarks

[ghost]

As Per The maintaining.md, I ran

$ node bench/run && node bench/compare

But, the result was

fs.js:928
  return binding.readdir(pathModule.toNamespacedPath(path), options.encoding);
                 ^

Error: ENOENT: no such file or directory, scandir 'E:\Workspace\ava\bench\.results'
    at Object.fs.readdirSync (fs.js:928:18)
    at Object.<anonymous> (E:\Workspace\ava\bench\compare.js:7:16)
    at Module._compile (module.js:649:30)
    at Object.Module._extensions..js (module.js:660:10)
    at Module.load (module.js:561:32)
    at tryModuleLoad (module.js:501:12)
    at Function.Module._load (module.js:493:3)
    at Function.Module.runMain (module.js:690:10)
    at startup (bootstrap_node.js:194:16)
    at bootstrap_node.js:666:3

[novemberborn]

As you found in #1755 the benchmarks have been broken for over a year… I never actually run them. Perhaps we should get rid of them. @avajs/core?

[sindresorhus]

I think the benchmark could be useful if we have performance-related commits or PRs. I would prefer to just fix them.

[ghost]

👍 to @sindresorhus

[OTR500miles2go]

I'd love to work on this if it's still an open issue.

[novemberborn]

@OTR500miles2go thanks for your interest! I don't mind it being fixed but it feels like wasted effort. It's 14 months since this issue was raised and I've still never tried to run benchmarks.

@sindresorhus?

[sindresorhus]

Yeah, let's just remove them.

[OTR500miles2go]

May I contribute by submitting a PR to:

• remove /bench
• update maintaining.md, remove the benchmark section
• update .gitignore, remove the bench/.results entry

[novemberborn]

Sounds great @OTR500miles2go!