Skip to content

Add binding.pry_shell to Lint/Debugger rule

Andrew Fontaine requested to merge afontaine/add-binding-pry-shell-cop into master

What does this MR do and why?

I noticed recently that Rubocop didn't warn that binding.pry_shell is a debugger method that should be removed, and I accidentally pushed a version of that commit with it included, but it should've been caught.

Here, I add it as one of the options for Lint/Debugger.

How to set up and validate locally

  1. Add binding.pry_shell to ruby file, run rubop
  2. Change to binding.pry, run rubocop to ensure original linter rules are caught

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports