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

Small fixes to calc tutorial #420

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

saona-raimundo
Copy link

Hi!

Excited about salsa, I was reading the calc tutorial in the book and thought that would be useful to contribute the small fixes I would make.

Most of the changes are typos, updating according to the code or rephrasing for a smoother reading flow. Feel free to accept or reject any of the changes: there is no such thing as the "best writing style".

Correct typos and make some sentences more explicit
Add anchor for `Default` implementation (through derive)
Correct some sentences from previous versions
Explicit some abbreviations.
Add some indications for better text flow.
Correct the heading levels.
Include some missing references in code.
Unify presentation style accross sections.
Change "id" for "struct" in interned context (for consistency)
This reads better in the tutorial, as the use statements will be part of the presented code
Correct typo.
Deduplicate text.
Update text to code.
Rephrase for better reading flow.
Fix typo
Source code documentation fixes
@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for salsa-rs ready!

Name Link
🔨 Latest commit d84a3eb
🔍 Latest deploy log https://app.netlify.com/sites/salsa-rs/deploys/633deabb0165ae0007249985
😎 Deploy Preview https://deploy-preview-420--salsa-rs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

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.

None yet

1 participant