search

nixme / pry-nav

Binding navigation commands for Pry to make a simple debugger
MIT License
390 stars 35 forks source link

Why is pry-nav discouraged for MRI? #41

Open justin808 opened 2 years ago

justin808 commented 2 years ago

Why is pry-nav discouraged for MRI?

Per the README:

Using MRI? We recommend break or pry-byebug instead!

pry-byebug has some issue with zeitwerk

iloveitaly commented 2 years ago

@justin808 honestly, not sure. Want to submit a PR to change this?

Most likely, https://github.com/ruby/debug is going to be the path forward, but can't hurt to update this for now.

nixme commented 2 years ago

Agreed with @iloveitaly that ruby/debug will be the path forward, but I originally put the recommendation because pry-nav uses set_trace_func which at the time was much slower than byebug. Ruby docs also say set_trace_func is obsolete. We could change to the TracePoint API and remove that recommendation.

justin808 commented 2 years ago

@iloveitaly Here you go: #42.