-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat: Quick Fix
for bare URLs
#12836
feat: Quick Fix
for bare URLs
#12836
Conversation
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.
I wanted to try this out in an IDE with rust-analyzer
but wasn't able to do so.
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.
I wanted to try this out in an IDE with rust-analyzer but wasn't able to do so.
not sure if there's a way 🤔, but you can put your test code in tests/ui/doc/doc-fixable.rs
, then run TESTNAME='doc-fixable' cargo uibless
to "try it out".
I'm surprised that this didn't have to touch any tests and the checks are still passing. We should definitely have a test for this like @J-ZhengLi said :) |
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.
Looks good to me, can you squash your commits? :)
Then we can merge this
2d45949
to
6f3e830
Compare
6f3e830
to
17cc0a3
Compare
Sure! I just squashed them after rebasing against the main branch. |
Thanks! @bors r+ |
You're welcome! |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
closes #12835.
Please write a short comment explaining your change (or "none" for internal only changes)
changelog: [
clippy::doc_markdown
]:Quick Fix
forbare URLs