Fix unused comment flag - #28
Conversation
|
👋 Hello @pderrenger, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
Clean PR. The diff only removes an unused inline assignment and reformats the existing re.sub() call without changing behavior. I did not find any actionable issues in the modified lines.
|
Merged — nice cleanup, @pderrenger, and thanks @glenn-jocher for the collaboration. As Leonardo da Vinci said, “Simplicity is the ultimate sophistication.” This small refactor is a great example of that: removing an unused inline assignment keeps Appreciate the thoughtful contribution and continued attention to code quality. |
Summary
Tests
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
🧹 This PR makes a small cleanup in
capitalize_comments.pyby removing an unused inline variable assignment while keeping behavior unchanged.📊 Key Changes
if is_comment := line.strip().startswith("#"):toif line.strip().startswith("#"):prev_was_commentbehavior is unchanged.🎯 Purpose & Impact