Warn about the risks of using catch-all patterns #200
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
This may not be obvious to people who have never experienced compiler-driven refactoring before, and feels worth calling out.
✅ Deploy Preview for taupe-lily-3539f6 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| Thanks! |
woojunro added a commit to woojunro/100-exercises-to-learn-rust that referenced this pull request Jan 2, 2025
* use OpenSans as fallback font on PRs from forks * Fix typo * Update reference from Rustonomicon to Rust Reference The exercise refers to the "Data layout" section of the Rustonomicon, but now that section claims itself deprecated and redirects to the "Type layout" section of The Rust Reference. * 05_ticket_v2 outro: correct comment on title (mainmatter#179) Characters was meant to be bytes, this was missed in a former refactoring * Change docstring: "when converted to `u8`" Fixed docstring * exercises 02_basic_calculator/04_panics: fix comma splice in panic message * Update 08_overflow.md if you read threw the profile docs it provides 4 profiles. https://doc.rust-lang.org/cargo/reference/profiles.html * chore(ci): re-create book artifacts at the beginning of each month * Update 08_overflow.md Added some context. * Update 11_clone.md (mainmatter#205) typo in code * format * debug pandoc build * fix reference to fallback font * switch off Pandoc verbosity * run Gravity on online PDF * debug * fix a superfluous word in a comment (mainmatter#217) * fix a small typo in 11_locks.md (mainmatter#219) * Fix typo in 03_leak.md (mainmatter#220) Corrected a confusing sentence with double negative. * Add install steps to the welcome page (mainmatter#196) * Warn about the risks of using catch-all patterns (mainmatter#200) * doc: update book 06_01_arrays (mainmatter#195) * Update aliases * Update redirects * Formatting * Link to the next tutored session in the methodology section * Ignore dead code warnings until visibility has been explained (mainmatter#232) * Format all manifests * Don't mark Rust code snippets as runnable * Showcase else-if (mainmatter#234) * Give a more structure hint for iteration exercises (mainmatter#235) * Clarify that both crate and super are keywords (mainmatter#236) * Clarify Send-Sync relationship (mainmatter#237) * Change if-else exercise to require an if-else (mainmatter#238) * Avoid using a suffix (mainmatter#239) --------- Co-authored-by: Marco Otte-Witte <marco.otte-witte@mainmatter.com> Co-authored-by: Simon Morgan <sjm@sjm.io> Co-authored-by: Luca Boschetto <61137723+LucaBoschetto@users.noreply.github.com> Co-authored-by: iacri <49108209+iacri@users.noreply.github.com> Co-authored-by: Henk Oordt <hd@oordt.dev> Co-authored-by: Benjamin Tamasi <half2me@users.noreply.github.com> Co-authored-by: Vincent Ging Ho Yim <cenviity@gmail.com> Co-authored-by: Manuel Bichler <bichler@gmail.com> Co-authored-by: Bartlomiej Dudzik <bartholomew.dudzik@gmail.com> Co-authored-by: Robertino <48034748+rober-m@users.noreply.github.com> Co-authored-by: dawe <dawedawe@posteo.de> Co-authored-by: Will Bush <git@willbush.dev> Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com> Co-authored-by: Maor Levy <Atixx@users.noreply.github.com> Co-authored-by: LukeMathWalker <20745048+LukeMathWalker@users.noreply.github.com>
colinstrout pushed a commit to colinstrout/100-exercises-to-learn-rust that referenced this pull request Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
This may not be obvious to people who have never experienced compiler-driven refactoring before, and feels worth calling out.