Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Command to toggle pry-state #4

Open
justin808 opened this issue Nov 8, 2015 · 8 comments
Open

Command to toggle pry-state #4

justin808 opened this issue Nov 8, 2015 · 8 comments

Comments

@justin808
Copy link

Super useful extension!

However, I'd like the ability to toggle this when debugging.

What do you think? @SudhagarS?

@SudhagarS
Copy link
Owner

Thanks @justin808.
I have plans to add some commands to have control over how values are displayed.
Think I will add a command for this too.

@justin808
Copy link
Author

@SudhagarS Sweet, let me know, and I'll try on the gem or directly from your repo. Check out my company's open source at https://github.com/shakacode/ and let me know if you need any help!

@kvokka
Copy link

kvokka commented Apr 16, 2016

Just few days ago I came to the same question.
You make so useful screen at the breakpoint, so, it will be great to be able to call it

@SudhagarS
Copy link
Owner

@kvokka I tried doing it few months ago but put it off as it required digging into pry. Anyway I will take a fresher look at it sooner and let you know.

@kvokka
Copy link

kvokka commented Apr 19, 2016

@SudhagarS Thank you. I've done the same yet. As variant, you may run binding.pry inside debugger session.

@matrinox
Copy link
Contributor

Any updates on this? I'm willing to help on this feature, just let me know what the parameters are (if any) and I'll get to it

@bbugh
Copy link

bbugh commented Jun 25, 2017

I have a fork with two commands:

  1. show-state which just shows the state of the current binding (as if it had been toggled)
  2. toggle-state which turns on and off the automatic display.

Since show-state is available as a command now, it made sense to me to turn off the hook by default. I also updated the README with information.

https://github.com/bbugh/pry-state/tree/state-commands#usage

When I have some time, I will write tests for the commands and changes, and submit a pull request.

@bbugh
Copy link

bbugh commented Jul 4, 2017

I added PR #10 for this, with test coverage now. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants