Skip to content

misc#754

Open
jstac wants to merge 1 commit into
mainfrom
mcmin
Open

misc#754
jstac wants to merge 1 commit into
mainfrom
mcmin

Conversation

@jstac
Copy link
Copy Markdown
Contributor

@jstac jstac commented May 28, 2026

@Chihiro2000GitHub ,

I made some modifications to your changes in

#741

Please have a look at the file and let me know if you have comments.

This is the intro lecture series, so we want to keep the code light and readable.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 28, 2026

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 92c106a
🔍 Latest deploy log https://app.netlify.com/projects/taupe-gaufre-c4e660/deploys/6a1896a0f7b28700089ce231
😎 Deploy Preview https://deploy-preview-754--taupe-gaufre-c4e660.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 project configuration.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 28, 2026

@github-actions github-actions Bot temporarily deployed to pull request May 28, 2026 19:38 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 28, 2026 19:38 Inactive
@Chihiro2000GitHub
Copy link
Copy Markdown
Contributor

Hi @jstac ,

Thank you very much for taking a close look, making these modifications, and reviewing my other PRs so carefully as well.

I may not be able to review this carefully right away, but I'll take a proper look early next week and let you know if I have any comments.

Thanks again!

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