Skip to content

Review linear_equations #263

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

Merged
merged 13 commits into from
Aug 16, 2023
Merged

Review linear_equations #263

merged 13 commits into from
Aug 16, 2023

Conversation

HengchengZhang
Copy link
Member

Hi @jstac, this PR applies the updates to linear_equations lectures.

Also thanks @DiaPorntipa and @orectique for the work done together, I've applied all the changes necessary so please have a look if you think there's still other changes needed.

@netlify
Copy link

netlify bot commented Jul 5, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 66a0329
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/64daf1021fce24000854a54d
😎 Deploy Preview https://deploy-preview-263--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 site configuration.

@github-actions
Copy link

github-actions bot commented Jul 5, 2023

@jstac
Copy link
Contributor

jstac commented Jul 5, 2023

Thanks @HengchengZhang , please see the reqeuested changes.

@HengchengZhang
Copy link
Member Author

HengchengZhang commented Jul 7, 2023

Hi @jstac, I've applied the requested changes.

There are also some minor updates.

@mmcky
Copy link
Contributor

mmcky commented Jul 31, 2023

@HengchengZhang would you mind resolving the conflicts and then letting me know when this is good to go?

@HengchengZhang
Copy link
Member Author

Hi @mmcky, I've resolved the conflicts and it should be good to go now.

@mmcky mmcky added the ready label Aug 7, 2023
@mmcky
Copy link
Contributor

mmcky commented Aug 15, 2023

@HengchengZhang just before I merge this -- can you add the index elements back into these documents?

orectique and others added 2 commits August 15, 2023 13:05
@HengchengZhang
Copy link
Member Author

@HengchengZhang just before I merge this -- can you add the index elements back into these documents?

Hi @mmcky I put them back in place.

@mmcky mmcky merged commit 3e283d6 into main Aug 16, 2023
@mmcky mmcky deleted the review_lectures branch August 16, 2023 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants