-
Notifications
You must be signed in to change notification settings - Fork 15.8k
Ruby : add content for debugging ruby using vscode rdbg extension #30267
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
base: main
Are you sure you want to change the base?
Conversation
…code rdbg debugger extension
…de using the vscode rdbg extension
JoshDevHub
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a big change--nearly 200 lines. Not necessarily a "bad" thing, it just makes me think that maybe we should go ahead and pull all stuff related to VSCode debugging into a new lesson. Can call it something like debugging_with_vscode. This was already the plan with #24994 anyway.
How does that sound? @XAJX179
Also going to suggest some other edits, mostly related to just nitpicky English things. I'll probably have more to add later as I need to sit with the lesson and run it through the lesson preview tool, but this will give you some work to do for now.
Co-authored-by: Josh Smith <[email protected]>
Sounds good! moved the content into it's own lesson. One thing left to do is updating image links after PR for images gets merged. Also since this is my first time adding a new lesson, i will have a look at the contributing guide tommorrow. |
Because
gaps in the current debug content.
This PR
adds a custom section heading with content on debugging with vscode-rdbg.
Issue
Closes #30210
Additional Information
Pull Request Requirements
location of change: brief description of changeformat, e.g.Intro to HTML and CSS lesson: Fix link textBecausesection summarizes the reason for this PRThis PRsection has a bullet point list describing the changes in this PRIssuesection