Skip to content

Update long_run_growth Lecture #342

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 9 commits into from
Feb 12, 2024
Merged

Update long_run_growth Lecture #342

merged 9 commits into from
Feb 12, 2024

Conversation

HumphreyYang
Copy link
Collaborator

This PR updates the lecture on long-run growth as discussed in the meeting today.

Copy link

netlify bot commented Jan 30, 2024

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 2596cdf
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/65c0b2f0401099000904aff1
😎 Deploy Preview https://deploy-preview-342--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.

Copy link

github-actions bot commented Jan 30, 2024

@jstac
Copy link
Contributor

jstac commented Feb 4, 2024

Thanks for putting this together @HumphreyYang , it's much appreciated.

There are many small comments but it would be great if you could make those adjustments and then spell check.

@HumphreyYang
Copy link
Collaborator Author

Many thanks @jstac,

This PR is in-work as I am also working on proofreading this morning. Your comments and reviews are just in time.

@HumphreyYang HumphreyYang requested a review from mmcky February 5, 2024 02:22
@HumphreyYang HumphreyYang added ready and removed in-work labels Feb 5, 2024
@jstac
Copy link
Contributor

jstac commented Feb 5, 2024

Thanks @HumphreyYang .

@mmcky Could you please do the next review and then coordinate with Tom regarding a good time to merge and make live?

Copy link
Contributor

@mmcky mmcky left a comment

Choose a reason for hiding this comment

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

thanks @HumphreyYang.

@mmcky
Copy link
Contributor

mmcky commented Feb 5, 2024

thanks @HumphreyYang I think the new sizes are good.

@jstac I think Long-run Growth looks funny as a title -- but is that the style we want? I am a bit capital happy.

@jstac
Copy link
Contributor

jstac commented Feb 5, 2024

@jstac I think Long-run Growth looks funny as a title -- but is that the style we want?

Yes, it should be "Long-Run Growth"

I am a bit capital happy.

:-)

@mmcky
Copy link
Contributor

mmcky commented Feb 8, 2024

thanks @HumphreyYang. I am coordinating with @thomassargent30 re: a good time to merge this.

@mmcky mmcky merged commit 5547cd4 into main Feb 12, 2024
@mmcky mmcky deleted the update-lr-growth branch February 12, 2024 05:13
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.

3 participants