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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lit CLI integration 馃敟 #2178

Closed
maedi opened this issue Feb 18, 2021 · 1 comment
Closed

Lit CLI integration 馃敟 #2178

maedi opened this issue Feb 18, 2021 · 1 comment

Comments

@maedi
Copy link

maedi commented Feb 18, 2021

Hi guys

I've created a little command line tool which helps with logging and debugging.

I have a "step" feature working where I can step through the code one log at a time by pressing the Enter key. At any step you are also able to "Press P for Pry" and enter a binding for the line of the log that you've stepped into. To do this I have to put a binding.pry in a block for the log method. Unfortunately this requires some runtime hackery to turn this:

lit("message")

Into this:

lit("message") { binding.pry if LitCLI.is_prying? }

Is there a better way to achieve this in Pry?

Ideally I wouldn't override require or require_relative, so that Lit can work out of the box with other frameworks that auto-require.

@andrehjr
Copy link
Member

andrehjr commented Mar 7, 2022

Hello @maedi

Not sure if I fully understand the issue. For pry to 'start' you need a 'binding' or an object for that context. You can call 'pry' on any object. See https://github.com/pry/pry/blob/master/lib/pry/core_extensions.rb#L24-L50

Does that help?

@andrehjr andrehjr closed this as completed Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants