-
Notifications
You must be signed in to change notification settings - Fork 13.3k
doc: bindings not needed for this example #31247
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
This is currently the style of the entire module. It would need to be consistently updated everywhere. I believe I chose this style because there are some examples which don't work with all of them, though I don't exactly remember. |
I see my style used in the module looking after a quick check. What are you looking at? |
The example directly below was what I double checked. When I did these docs, I took a lot of pains to make them all consistent with each other generally. I'm not perfect though :) |
The example below is just a copy. Nowhere else in the module did I see the style. |
Okay, I double checked myself, because I swore I did it this way for all of them. It turns out that this set and Would you mind updating the second block here to use this style, and Sorry about mis-remembering :( |
Thanks for coming back to this issue |
@bors: r+ rollup |
📌 Commit 4289973 has been approved by |
No description provided.