skip if ThreadClient.current.management?

[ko1]

fix #609

[st0012]

I think it's great to centralize skipping logic but I don't think skip_path? is the place for it. Maybe we can have rename SkipPathHelper to SkipHelper and add something like skip? to it? So it'll be

skip?(path)

module SkipHelper
  def skip?(path)
    ThreadClient.current.management? || skip_path?(path)
  end
end

[ko1]

Good idea. Another PR.