Update README to mention JRuby and fix circular require warning

nixme / pry-nav

Binding navigation commands for Pry to make a simple debugger

MIT License

390 stars 35 forks source link

Update README to mention JRuby and fix circular require warning #26

Closed ivoanjo closed 8 years ago

ivoanjo commented 8 years ago

This PR:

updates the README to always recommend pry-byebug for MRI users (as you should know better if you're still on < 1.9)
cleans up a circular require warning
bumps the gem version to 0.2.5

nixme commented 8 years ago

Also, can we close #19 if your changes to jRuby mean it's fixed?

ivoanjo commented 8 years ago

Yeah, I think #19 can be closed too. The -Xcext.enabled=true option also being discussed there I'm not sure it was ever needed but I can confirm that it's not needed right now, nor really supported in JRuby 9k.

jasonrclark commented 6 years ago

Any chance of a gem release with this change in it? I'm using JRuby, so can't do pry-byebug and I'm trying to run tests with warnings on, but this spams the output a lot. Thanks!

ivoanjo commented 6 years ago

@jasonrclark unfortunately the author has been quite irresponsive to my communication attempts and other PRs.

My plan was to push improvements upstream but I ended up working on a fork of @nixme's other debugging gem: https://github.com/ivoanjo/pry-debugger-jruby .