QuantEcon / lecture-python-programming.myst

Python Programming for Finance and Economics
https://python-programming.quantecon.org
43 stars 26 forks source link

LECTURE: Development Environments Updated and Added #283

Closed orectique closed 9 months ago

orectique commented 1 year ago

fixes #279

github-actions[bot] commented 1 year ago

🚀 Deployed on https://6577d25656128f7bc9b6e0cd--epic-agnesi-957267.netlify.app

jstac commented 1 year ago

Thanks @orectique . Could you please look into the failing link check?

orectique commented 1 year ago

I've verified that the links flagged by the link checker are all functional.

It might be that the flagged sites are shutting down automated access.

jstac commented 1 year ago

@mmcky or @HumphreyYang , could you please advise? I think we just ignore the link checker in this case?

HengchengZhang commented 1 year ago

Hi @jstac, @mmcky and @HumphreyYang, I checked these links manually and they all should be false positives. My opinion is to add them to the ignore list.

HumphreyYang commented 1 year ago

Hi @jstac, @mmcky and @HumphreyYang, I checked these links manually and they all should be false positives. My opinion is to add them to the ignore list.

Many thanks @HengchengZhang and @jstac. I also think we can have it on since it still works on other links. We can manually check the white list when we have major releases as long as the list is not too long. We can consider disabling it in the future if the list of false positives is getting too long.

jstac commented 1 year ago

Thanks @HengchengZhang and @HumphreyYang . Let's ignore as suggested.

jstac commented 1 year ago

@orectique Nice work on this lecture. Let's meet up and discuss it. I'll contact you to set up a time.

HengchengZhang commented 1 year ago

Hi @jstac I added them to the ignore list and it's working now.

orectique commented 1 year ago

Thanks, @jstac.

mmcky commented 1 year ago

thanks @orectique -- this looks like it is in-work but let me know if you need a review etc.

orectique commented 1 year ago

@mmcky This needs a review, please. I've added new content to the Getting started lecture and revised the workspaces lecture.

mmcky commented 12 months ago

Sorry @orectique this slipped through my radar. Can you update on the latest main and then ping me for review. Thanks,.

orectique commented 11 months ago

Hi @mmcky,

Thanks for the review. I have merged changes on Main to this branch. There should be no conflicts.

mmcky commented 9 months ago

thanks @orectique I reviewed 95de21e and see they resolved all the outstanding items. Thank you.

mmcky commented 9 months ago

thanks @orectique -- @jstac this is now at review stage.

jstac commented 9 months ago

Thanks @mmcky , looks good to me. It's nicely written.

mmcky commented 9 months ago

thanks @jstac and @orectique for these improvements.