Skip to content

Exclude a few more directories for rubocop

Ash McKenzie requested to merge ashmckenzie/speed-up-rubocop into main

What does this Merge Request do and why?

This MR improves the speed/performance of rubocop by excluding a few more directories that it shouldn't parse. On my machine, it's a big improvement:

Before

$ time make rubocop
RuboCop: Inspecting 170 files
..........................................................................................................................................................................

170 files inspected, no offenses detected
make rubocop  54.73s user 4.95s system 88% cpu 1:07.65 total

After

$ time make rubocop
RuboCop: Inspecting 170 files
..........................................................................................................................................................................

170 files inspected, no offenses detected
make rubocop  11.07s user 3.36s system 186% cpu 7.729 total

Merge Request checklist

  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Tests added for new functionality. If not, please raise Issue to follow-up.
  • Documentation added/updated, if needed.
  • gdk doctor test added, if needed.
  • Add the ~highlight label if this MR should be included in the CHANGELOG.md.

Merge request reports