Skip to content
This repository has been archived by the owner on Mar 23, 2020. It is now read-only.

Add english Syntax chapter #60

Merged
merged 2 commits into from
May 11, 2017
Merged

Add english Syntax chapter #60

merged 2 commits into from
May 11, 2017

Conversation

Hoverbear
Copy link
Member

Contributes to #28 .

@skade
Copy link
Collaborator

skade commented May 11, 2017

I would put "if let" after match.

Copy link
Collaborator

@skade skade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine, with the nit that I'd prefer "if let" to be explained after or with match.

@Hoverbear
Copy link
Member Author

@skade Would you suggest putting if and if let after, or giving if let it's own (third) slide?

@skade
Copy link
Collaborator

skade commented May 11, 2017

I would give if let a separate slide, as I consider it a special form of pattern matching.

@Hoverbear
Copy link
Member Author

Our opinions differ, but I'll go with your preference!

@Hoverbear Hoverbear merged commit 50fedd4 into master May 11, 2017
@Hoverbear Hoverbear deleted the feature/syntax branch May 11, 2017 12:30
gaconkzk pushed a commit to gaconkzk/learn-rust that referenced this pull request Jun 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants