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

Initialize the English lecture 4 #3

Merged
merged 38 commits into from
Mar 26, 2024
Merged

Initialize the English lecture 4 #3

merged 38 commits into from
Mar 26, 2024

Conversation

Milky2018
Copy link
Contributor

No description provided.

Copy link
Contributor

@skylee03 skylee03 left a comment

Choose a reason for hiding this comment

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

This is lecture 4, not lecture 7.

@peter-jerry-ye peter-jerry-ye changed the title Initialize the English lecture 7. Initialize the English lecture 4 Mar 12, 2024
Copy link
Collaborator

@peter-jerry-ye peter-jerry-ye left a comment

Choose a reason for hiding this comment

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

The overall result is great, thank you. There are just some minor comments from me.

Milky2018 and others added 4 commits March 12, 2024 11:34
Co-authored-by: Yang Ming-Tian <1178715749@qq.com>
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
Update the content verified in conversations.
@Milky2018
Copy link
Contributor Author

Thanks for reviewing.

I have updated this lecture with 4 new commits.

Comment if you have any other suggestions. 0v0

Add an example for pattern matching in let bindings.
@peter-jerry-ye
Copy link
Collaborator

Nice job. Please continue with the slides, and I'll review them together.

Copy link
Collaborator

@peter-jerry-ye peter-jerry-ye left a comment

Choose a reason for hiding this comment

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

LGTM

@peter-jerry-ye peter-jerry-ye requested a review from bobzhang March 21, 2024 02:13
Milky2018 and others added 6 commits March 21, 2024 13:14
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
@peter-jerry-ye peter-jerry-ye removed the request for review from bobzhang March 25, 2024 07:56
@peter-jerry-ye
Copy link
Collaborator

Coud you please do a rebase?

Milky2018 and others added 13 commits March 25, 2024 16:21
Co-authored-by: Yang Ming-Tian <1178715749@qq.com>
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
Update the content verified in conversations.
Add an example for pattern matching in let bindings.
Replace `\{` with `\lbrace`
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
Co-authored-by: Zihang Ye <jerryzihye@outlook.com>
@peter-jerry-ye
Copy link
Collaborator

Maybe you can add some no-check annotation to disable some checks for now

@peter-jerry-ye
Copy link
Collaborator

LGTM

@peter-jerry-ye peter-jerry-ye merged commit 2ad9e0b into moonbitlang:main Mar 26, 2024
1 check passed
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.

3 participants