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

[HACKATHON] Enable user narrative script to work with binding.pry #11651

Open
wants to merge 3 commits into
base: login-hackathon-2024-user-narrative
Choose a base branch
from

Conversation

eileen-nava
Copy link
Contributor

🛠 Summary of changes

  • I added a command line flag, -f, for passing in a file that contains ndjson. I made this change to enable using binding.pry with the script. Previously, the script regarded the piped-in file as stdin, which made it impossible to interact with binding.pry breakpoints via the terminal.
  • Since gpo was ripped out of the ipp flow, I removed the handling of the ipp + gpo flow.

📜 Testing Plan

  • Drop a binding.pry somewhere in the script.
  • Run the script and interact with the breakpoint via the CLI.

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

Successfully merging this pull request may close these issues.

1 participant