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

Add classes cheatsheet to classes lesson #495

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hroncok
Copy link
Member

@hroncok hroncok commented Nov 28, 2018

Done from browser, untested.

courses/pyladies/info.yml Outdated Show resolved Hide resolved
Co-Authored-By: hroncok <[email protected]>
@hroncok
Copy link
Member Author

hroncok commented Dec 15, 2018

BTW this is awesome (not applicable here):

_

@encukou
Copy link
Member

encukou commented Dec 15, 2018

There's something vague about the classes cheatsheet that worries me, but I can't seem to put it into words properly. Can we discuss in person (so I can either find the right words to document it for everyone, or see that I shouldn't be worried)?

@encukou
Copy link
Member

encukou commented Mar 7, 2019

We finally met about this!
The cheatsheet needs a few improvements:

  • The arrow to snez should go to Prasatko
  • Typová kontrola – change variable to prasatko, remove from the loop
  • Make the explanations in the right column not depend on each other (where possible)
  • Kotatko.vysun_drapky instead of zamnoukej; remove zakvic
  • The Polymorphism section can be removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants