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

Include debug gem #4215

Merged
merged 2 commits into from
Mar 20, 2024
Merged

Include debug gem #4215

merged 2 commits into from
Mar 20, 2024

Conversation

Crashillo
Copy link
Member

@Crashillo Crashillo commented Mar 20, 2024

✌️ What does this PR do?

Include the new ruby's debug gem. It works alongwith puma puma-dev -sysbind (instead of rails server), through rdgb --attach CLI command.

https://world.hey.com/lewis/run-multiple-rails-apps-with-puma-dev-67b1c10f
https://github.com/ruby/debug
https://gobierto.readme.io/docs/development-environment-1#debugging-backend-with-puma-dev

@Crashillo Crashillo requested a review from ferblape March 20, 2024 09:08
@ferblape
Copy link
Member

Could you add those steps also in the README?

@Crashillo
Copy link
Member Author

Could you add those steps also in the README?

Where exactly? This README is different from others... and veeeery dated 😅

@ferblape
Copy link
Member

ferblape commented Mar 20, 2024 via email

@Crashillo
Copy link
Member Author

Yes, I can edit right in the docs. No problem

@Crashillo
Copy link
Member Author

@ferblape
Copy link
Member

Just remove byebug from the Gemfile, run bundle install and we are ok to merge

@Crashillo
Copy link
Member Author

Crashillo commented Mar 20, 2024

Ah, I left there just in case someone, with no puma-dev setup, would want to use the legacy byebug gem, as still working. But OK, maybe is too much to have two different gems.

@ferblape ferblape merged commit 200cc48 into master Mar 20, 2024
17 of 19 checks passed
@ferblape ferblape deleted the debug-gem branch March 20, 2024 14:48
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.

None yet

2 participants