Skip to content
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

Some Ideas for "Extending" Rust Teaching #140

Closed
wants to merge 2 commits into from

Conversation

shawntabrizi
Copy link

@shawntabrizi shawntabrizi commented Sep 21, 2023

My high level comment about the Rust content so far, is that it basically re-hashes the same material that is taught in the Rust book, and can be learned most anywhere.

I think what we really want to do is not reteach what is already out there, but simply extend knowledge and best practices in the context of Substrate / Blockchain development, and the content that are learning.

These examples do require that users have already some previous knowledge, but there is certainly a way to fit this content in.

Perhaps as a second journey through learning Rust.

For each Rust section, I want to ask, and document: "What are some specific / advance tips we can give using this information in the context of Blockchain development.

This is just one part of one section which kind of shows the direction I am thinking.

cc @DrW3RK @CrackTheCode016

@CrackTheCode016
Copy link
Contributor

I like this flow. I think most basic concepts can be covered in this manner, provided they flow into each other naturally.

A simple, yet powerful example is proper error handling. Even based on the content above, it would be easy to start thinking about defining custom errors, introducing things like the Default trait, handling them properly, and emphasizing a strict no panic policy. From here, it would be far simpler to start thinking about abstracting the codebase, then separating into modules, files, etc.

Condensing these basic things into one or two modules leaves more room for more advanced content. I left a comment with some suggestions on your tracking issue #134 (I didn't see this PR before!) that goes into more depth on some of the content that could be emphasized.

@shawntabrizi
Copy link
Author

closing this because it is stale

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants