1
0
mirror of https://github.com/google/comprehensive-rust.git synced 2025-06-15 13:50:27 +02:00
Commit Graph

715 Commits

Author SHA1 Message Date
61aa12e3f3 Merge pull request #27 from taufik-rama/docs/rustup-installation
Mention rustup.rs on the installation page

Fixes #19.
2022-12-23 11:41:30 +01:00
1b59aa5d71 Merge pull request #45 from Arthur-Milchior/patch-3
Makes ownership of `self` more accurate
2022-12-23 11:39:08 +01:00
952e778527 Merge pull request #43 from reillyeon/patch-3
Grammar fix in ownership/lifetimes-function-calls.md
2022-12-23 11:34:51 +01:00
21a2bb235f Merge pull request #41 from reillyeon/patch-1
Fix C array syntax in memory-management/manual.md
2022-12-23 11:27:15 +01:00
71caeae674 Merge pull request #39 from dzvon/patch-1
Fix a typo
2022-12-23 11:25:00 +01:00
bbc98a44ec Merge pull request #25 from kianmeng/misc-doc-fixes
Misc doc fixes
2022-12-23 11:19:45 +01:00
b9085905f1 Simplifies rustup 2022-12-23 17:11:24 +07:00
3631117446 Makes ownership of self more accurate
It’s entirely possible that a method gives ownership of self to another object, or return `self`. 
For example
```
    fn id(self) -> Self {
        return self;
    }
```
or more realistically 
```
    fn or(self, other) -> Self {
      if (self.isValid()) { return self;}
      return other;
    }
```
2022-12-23 00:16:42 -08:00
73e9a0b854 Update references-dangling.md 2022-12-23 15:41:41 +08:00
9464def62e Grammar fix in ownership/lifetimes-function-calls.md 2022-12-22 22:09:17 -08:00
5223ef8f25 Change "semantics is" to "semantics are"
Subject-verb agreement.
2022-12-22 22:05:38 -08:00
16d4d2c92f Fix C array syntax in memory-management/manual.md
Heap-allocated arrays in C use `int*` instead of `int[]`.
2022-12-22 21:58:08 -08:00
54adbe7f55 Misc doc fixes
List of changes:
- fix typos found via `codespell -S target -L crate`
- ignore Cargo.lock
2022-12-23 11:24:51 +08:00
0ebfc8a54a docs: fix a typo 2022-12-23 10:36:58 +08:00
264047445f Merge pull request #28 from zdimension/patch-2
Add footnote about memory leaks
2022-12-22 20:29:10 +01:00
4126cbd519 Tuples with "..." make no sense, so don't use them 2022-12-22 20:25:15 +01:00
06b594b47a Merge pull request #26 from mxschumacher/patch-1
Fix typo
2022-12-22 20:18:54 +01:00
3bed28a9d9 Add "mostly" 2022-12-22 20:18:36 +01:00
aa12047dce Merge pull request #33 from LLyaudet/patch-1
Corrected 1 or 2 typos
2022-12-22 19:59:06 +01:00
466b6e9cd9 Merge pull request #30 from google/mdbook-versions
Include version numbers when installing `mdbook`
2022-12-22 19:57:22 +01:00
b29dd1ae09 Merge pull request #31 from google/mdbook-test
Add a build workflow to test PRs
2022-12-22 19:56:54 +01:00
1315481e69 Corrected 1 or 2 typos
missing verb :
"The goal of the course to teach you Rust." -> "The goal of the course is to teach you Rust."

missing of ? :
"to cover all it" -> "to cover all of it"
for this one, I'm not a native english speaker but it seems more correct to me. I'll let you check :)
2022-12-22 18:33:02 +01:00
11789caee2 Add a build workflow to test PRs
We will expand this to build/test other things in the future.
2022-12-22 15:22:41 +01:00
85c1684fe3 Include version numbers when installing mdbook 2022-12-22 15:01:55 +01:00
5dd366d7db Remove part about Rc 2022-12-22 14:24:47 +01:00
7b7b56398e Add footnote about memory leaks 2022-12-22 14:03:18 +01:00
b9eed2a072 Minor sidenote for rustup installation as an alternative to distro package manager 2022-12-22 19:49:58 +07:00
15b8ca8a05 Merge pull request #22 from Arthur-Milchior/patch-1
Clarify from/into
2022-12-22 13:23:55 +01:00
adc64c020b fix typo 2022-12-22 11:52:44 +01:00
121a7d33f3 Merge pull request #7 from andrewpollack/patch-5
Minor punctuation update
2022-12-22 10:08:16 +01:00
0e74800885 Merge pull request #9 from andrewpollack/patch-7
Minor punctuation addition
2022-12-22 10:07:58 +01:00
7a215992b4 Minor typo, punctuation update (#5)
* Minor typo
2022-12-22 10:07:26 +01:00
d66506fe35 Merge pull request #3 from andrewpollack/patch-1
Adding colon punctuation to lists
2022-12-22 10:06:15 +01:00
1a7df6cfb1 Merge pull request #6 from andrewpollack/patch-4
Minor punctuation update
2022-12-22 10:05:53 +01:00
ed0a11a86b Merge pull request #10 from andrewpollack/patch-8
Minor punctuation update
2022-12-22 10:05:08 +01:00
d80b91b3fd Merge pull request #20 from lonesometraveler/typo
typo: assigment -> assignment
2022-12-22 10:03:35 +01:00
b424abf704 Merge pull request #14 from google/gh-action-cache
Cache the artifacts built by the publish workflow
2022-12-22 09:45:16 +01:00
638dde7704 Clarify from/into
I found the phrasing confusing. Mentionning that a class implements From and then using `into()` seems confusing. It looks like that you meant that it implements `Into`.
So I believe that a little more details may help show why this exercise is valid.

I would have considered clarifying that `into` does not need to specify that we use the implementation for `i16` thanks to type analysis, but I assume this would add too much text, and this can either be ignored by the reader for the time being, or guessed
2022-12-21 16:46:23 -08:00
62f530d5ac typo: assigment -> assignment 2022-12-21 19:45:02 -05:00
b411aa688a typo: hygenic -> hygienic 2022-12-21 21:43:53 +00:00
a6b5c7171d grammar/typo: it's -> its 2022-12-21 21:42:58 +00:00
a96f48456d Merge pull request #15 from jrvanwhy/unsafe-skip-details
`welcome.md`: indicate we skip the subtle details of unsafe Rust
2022-12-21 19:19:20 +01:00
24283c9806 Merge pull request #4 from andrewpollack/patch-2
Suggestion: link to Cargo book on introduction
2022-12-21 19:03:43 +01:00
d4655aae9d Merge pull request #13 from andrewpollack/patch-10
Minor punctuation update
2022-12-21 19:03:19 +01:00
8f60685165 welcome.md: indicate we skip the subtle details of unsafe Rust
This sentence was missing a negation: it was supposed to say this course does not cover unsafe Rust in depth, but actually said "we go into the subtle details". This PR adds the missing negation.
2022-12-21 10:01:11 -08:00
653a818b8b Cache the artifacts built by the publish workflow
This should speed up the publication: right now we spend around 4
minutes to build `mdbook` and `mdbook-svgbob`.

With this change, we will only pay that cost once a week (caches
created by GitHub Actions last 7 days[1]).

[1]: https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#usage-limits-and-eviction-policy
2022-12-21 18:49:41 +01:00
d752b11694 Minor punctuation update (#11)
* Minor punctuation update

* Update sync.md
2022-12-21 17:41:58 +00:00
19ec43b8ec Merge pull request #12 from google/link-to-material
Add link to course material
2022-12-21 18:40:02 +01:00
ebd33a5b5f Merge pull request #8 from andrewpollack/patch-6
Add link to FizzBuzz definition
2022-12-21 18:39:49 +01:00
e203586182 Minor punctuation update 2022-12-21 09:38:05 -08:00