Skip to content
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

Clarify that r? works in comments. #1289

Merged

Conversation

iwanders
Copy link
Contributor

Reading the docs here I thought r? only applied to the original PR description. @nagisa pointed out out that it should work in comments, so adding that information to the guide seems relevant as it currently only talks about the initial description.

src/contributing.md Outdated Show resolved Hide resolved
@camelid camelid added the S-waiting-on-author Status: this PR is waiting for additional action by the OP label Jan 21, 2022
@camelid
Copy link
Member

camelid commented Jan 21, 2022

Thanks! I left one comment, otherwise this looks good.

@iwanders iwanders force-pushed the clarify-r-questionmark-works-in-comments branch from 03cc4ef to 68785e4 Compare January 21, 2022 01:30
Copy link
Member

@camelid camelid left a comment

Choose a reason for hiding this comment

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

Thanks again!

@camelid camelid merged commit 341abec into rust-lang:master Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: this PR is waiting for additional action by the OP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants