Skip to content

Conversation

@despadam
Copy link
Contributor

Addresses: #282, #244

@despadam despadam linked an issue Oct 31, 2025 that may be closed by this pull request
@despadam despadam self-assigned this Oct 31, 2025
Copy link
Member

@yakutovicha yakutovicha left a comment

Choose a reason for hiding this comment

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

Great work, @despadam. Just a bunch of minor suggestions.

Co-authored-by: Aliaksandr Yakutovich <yakutovicha@gmail.com>
Copy link
Collaborator

@Snowwpanda Snowwpanda left a comment

Choose a reason for hiding this comment

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

Great content! Good work @despadam , i hope this will help many people, and we can get feedback from ppl after the first session. I have some smaller mostly optional changes.

Copy link
Member

@edoardob90 edoardob90 left a comment

Choose a reason for hiding this comment

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

Very much complete: thorough but not "too much" for complete beginners. Thanks @despadam! I've only two quite small suggestions.

@despadam despadam requested a review from fabioacl November 5, 2025 12:25
@edoardob90 edoardob90 self-requested a review November 5, 2025 12:25
edoardob90
edoardob90 previously approved these changes Nov 5, 2025
Copy link
Member

@edoardob90 edoardob90 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Copy link
Contributor

@fabioacl fabioacl left a comment

Choose a reason for hiding this comment

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

Only some minor details and it is good to go. Very good job @despadam. I really liked the last example where you use all the concepts you introduced before.

Co-authored-by: Fábio Lopes <33908066+fabioacl@users.noreply.github.com>
@despadam despadam dismissed stale reviews from Snowwpanda and yakutovicha November 6, 2025 12:21

Already addressed.

Copy link
Member

@yakutovicha yakutovicha left a comment

Choose a reason for hiding this comment

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

Great job, @despadam!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create an introductory notebook about syntax

6 participants