Skip to content
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

Pyladies course update #48

Closed

Conversation

petracihalova
Copy link

update of the basic beginner's python course (for self-study) available here: https://naucse.python.cz/course/pyladies/

  • for each lesson, materials are split into sections (learning materials, recordings, additional resources)
  • added recordings for many lessons (source: encukou's youtube playlist)
  • updated course long_description
  • pyglet lesson moved in the end of the course
  • added additional final projects (Snake + Klondike)
  • added lesson about VS Code (created for Ostrava 2022 run https://naucse.python.cz/2022/ostrava-python/beginners/vscode/)
  • added lesson about JSON and API

@encukou
Copy link
Member

encukou commented Oct 22, 2024

Could you split this into multiple PRs?

For the videos, yes, I guess it's time to add them. The sections make sense (they could use better support in naucse_render, but, let's not make that block the addition).

Git, and Graphics+games, would perhaps work better as separate courses. For now, moving them so they're together makes sense.

For JSON/API, the example app for the old lesson is offline, and for GitHub we should check if things didn't change. Let's not merge as-is.

For VS Code, I also don't think it's ready as-is. There are some typos, and it needs to be adjusted & checked on macOS & Linux. Having it in a separate PR would help. But, since not all cities use VS Code, perhaps keep it in Ostrava for a while longer.

@petracihalova
Copy link
Author

@encukou thank you for feedback, I will close this one and prepare new PRs

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