Skip to content

Conversation

@XAJX179
Copy link
Contributor

@XAJX179 XAJX179 commented Oct 20, 2025

Because

Need images for the content in the PR #30267

This PR

  • adds images to ruby/basic_ruby/debugging/img/ folder.

Issue

Related to #30210

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

Copy link
Member

@KevinMulhern KevinMulhern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I'm just seeing I was assigned to this!

The images look good to me @XAJX179! But I'd also like to get @JoshDevHub's blessing on this before merging since he's reviewed the meat of the changes in the other PR, and can judge this much better than me.

Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These look good!

Just one request: since we moved the new content you're adding to its own lesson, can you change the dir name that these images will exist in?

# right now
ruby/basic_ruby/debugging/imgs/<img>.png

# should be->
ruby/basic_ruby/debugging_with_vscode/imgs/<img>.png

@XAJX179 XAJX179 requested a review from JoshDevHub November 25, 2025 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content: Ruby Involves the Ruby course

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants