fix: correct www autofix in no-bare-urls - #720
tooth-is-silver wants to merge 8 commits into
Conversation
no-bare-urls
lumirlumir
left a comment
There was a problem hiding this comment.
Disclosure: I'm a participant of open source contribution program OSSCA: confirmed.
Thanks for the PR. I’ve left a few comments about incorrect autofix cases.
|
Hi @tooth-is-silver, are you still working on this? |
Hi @snitin315 ! |
|
@lumirlumir, could you please take a look at the updated commits? |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe ChangesWWW autolink autofix
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~12 minutes Change: Bug fix · Severity of issue fixed: Medium Suggested reviewers: Merge Risk: ⚪ Minimal · up to The www autolink fix produces an explicit Markdown link rather than the crashing autofix form, with regression coverage for the intended behavior. No actionable merge risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Prerequisites checklist
AI acknowledgment
What is the purpose of this pull request?
Fix the incorrect autofix for
wwwautolinks.What changes did you make? (Give an overview)
Updated
no-bare-urlsto convertwww.example.cominto[www.example.com](http://www.example.com).Added a test for the autofix result.
Related Issues
Fixes #710
Is there anything you'd like reviewers to focus on?
Nothing in particular.
Disclosure: I'm a participant of open source contribution program OSSCA
Summary by CodeRabbit
www.links as Markdown links instead of angle-bracket autolinks.