Replace byebug with debug for debugging#364
Open
larouxn wants to merge 3 commits intoShopify:mainfrom
Open
Conversation
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
a665dc4 to
b992b67
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
2 times, most recently
from
72098e2 to
bd617a1
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
bd617a1 to
9b285a5
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
9b285a5 to
b6207a8
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
b6207a8 to
659cdff
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
659cdff to
b12a435
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
b12a435 to
f4ca7b1
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
3 times, most recently
from
4a68162 to
01144f0
Compare
1 task
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
01144f0 to
97677b3
Compare
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
97677b3 to
ff95290
Compare
- Add no-longer-stanadard-library gems to Gemfile and Gemspec - Upgrade gems to versions that support Ruby 4 - Add Ruby 4 to CI matrix - Bump dev Ruby from 3.4.8 to 4.0.1
larouxn
force-pushed
the
replace_byebug_with_debug
branch
from
ff95290 to
61af2a3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What are you trying to accomplish?
Replace the use of
byebug(plusirbandreline) withdebugfor debugging.Follow up to
#359 and #363#362 and #434What approach did you choose and why?
Cleaner, simpler, and more modern.
What should reviewers focus on?
Any reason we want to stick with byebug?
The impact of these changes
More standard Ruby codebase.
Testing
Testing debugging works while working on fixing these tests.
Checklist
Determined not necessary as it's a developer facing change, not user facing.