Conversation
ehuss
left a comment
There was a problem hiding this comment.
Each code block should use the appropriate language. text disables highlighting and better conveys the intent of what the code block is for.
| First, let’s take a look at the directory structure of this package: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```text |
| Like above, let’s first take a look at the package layout: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```text |
| layout of this scenario: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```text |
| actively denying third party access. To check this, you can go to: | ||
|
|
||
| https://github.com/organizations/:org/settings/oauth_application_policy | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```text |
| environment variable will accomplish the same thing as defining a config file: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```text |
| Example: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```sh |
| component: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```console |
| Usage looks like: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```console |
| of these crates, then you can pass an argument to `-Zbuild-std` as well: | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```console |
| takes, and tracks concurrency information over time. | ||
|
|
||
| ``` | ||
| ```ignore |
There was a problem hiding this comment.
| ```ignore | |
| ```sh |
|
☔ The latest upstream changes (presumably #7823) made this pull request unmergeable. Please resolve the merge conflicts. |
|
I'm going to close, as I haven't heard back in a while. Feel free to reopen with the suggested changes, or open new PR if you're still interested. |
fix mdbook test with ```ignore/text/sh/console I revitalized the dead PR #7832 sorry about the delay testing is just running `mdbook test` which now passes! Let me know if you want me to add mdbook test to ci as well!
The rust reference repo has
mdbook testas part of its CI, so when I was working on a PR here,mdbook testbeing broken confused me, so I figured I would make a PR to fix it.I added some ignores to the problematic blocks (basically those with dependencies or using env vars)
and fixed some links to use