Add binding.pry_shell to Lint/Debugger rule
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
- Add
binding.pry_shell
to ruby file, run rubop - 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.
-
I have evaluated the MR acceptance checklist for this MR.