-
Notifications
You must be signed in to change notification settings - Fork 216
Move most of the README into the Guide. #334
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
|
See the updated README here. |
3932f85 to
278c0a3
Compare
LegNeato
left a comment
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 so much better!
README.md
Outdated
| ## Documentation | ||
|
|
||
| A sample `.devcontainer.json` file is also included, configured for Ubuntu 24.02. Copy this to `.devcontainer/devcontainer.json` to make additional customizations. | ||
| Please see [The Rust CUDA Guide](https://rust-gpu.github.io/rust-cuda/index.html) for all the |
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.
nit: I'd remove the index.html to be cleaner
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.
Done
It annoys me to have stuff split between the README and the Guide. This commit moves most of the README contents into the Guide's introduction, (which was all but empty anyway). I also did some line editing of the moved text. Other minor structural changes: - Rename the first chapter file from `README.md` (not to be confused with the top-level README) to `introduction.md`, to match the chaper name. - Move "Supported Features" and the FAQ to unnumbered chapters at the end -- making them quasi-appendices -- because they're supplementary. This means the "Getting Started" section now follows immediately on from the introduction, which is good.
Something I learned recently.
This showed up in Rust-GPU#220 and Rust-GPU#327.
This file currently uses a mix of 2-space and 4-space indents in examples.
Currently they don't show up. They need to be within the `src/` directory to get included in the generated output.
80decdd to
afe3fe2
Compare
|
New version is up! |
It annoys me to have stuff split between the README and the Guide. This commit moves most of the README contents into the Guide's introduction, (which was all but empty anyway). I also did some line editing of the moved text.
Other minor structural changes:
README.md(not to be confused with the top-level README) tointroduction.md, to match the chaper name.