Skip to content

Replace byebug and pry

Takuya Noguchi requested to merge tnir-replace-byebug-with-debug into master
Replace byebug and pry

- Replace byebug with ruby/debug
- Replaces pry with irb

Closes https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/6955

Closes https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/6956

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>

Changelog: other

What does this MR do?

Replaces byebug gem (a debugger for Ruby 2) with debug gem (also known as ruby/debug) (default debugger for Ruby 3). Also replaces pry gem with irb gem, the default gem in Ruby 3.0 or higher.

Note 1: pry was introduced in !3217 (merged), but no reference there.

Note 2: pry is still depended from inspec-core, which is not easy to remove it at all.

Related issues

Closes #6956 (closed)

Also closes #6955 (closed)

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion

Required

  • Merge Request Title, and Description are up to date, accurate, and descriptive
  • MR targeting the appropriate branch
  • MR has a green pipeline on GitLab.com
  • [n/a] Pipeline is green on dev.gitlab.org if the change is touching anything besides documentation or internal cookbooks
  • trigger-package has a green pipeline running against latest commit

Expected (please provide an explanation if not completing)

  • [n/a] Test plan indicating conditions for success has been posted and passes
  • [n/a] Documentation created/updated
  • [n/a] Tests added
  • [n/a] Integration tests added to GitLab QA
  • [n/a] Equivalent MR/issue for the GitLab Chart opened
Edited by Robert Marshall

Merge request reports