-
Notifications
You must be signed in to change notification settings - Fork 120
[WIP] Add HTML versions of homework #153
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The main problem I see here is that there's no way to work with the individual tests – for example, to attach submission forms to them. I think a bit more infrastructure is needed to represent the individual tasks.
import asyncio | ||
import contextlib | ||
import time | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
intro/async/static/actor.py
was removed since it's no longer used. Please don't re-add it.
a zjisti, kolikrát je v něm použito písmeno K.<br> | ||
<em>Připomínám: Až to bude fungovat, dej to do Gitu!</em> | ||
</li> | ||
</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra </li>
materials/homework/file/index.html
Outdated
původní jméno a nové jméno, načte původní soubor a zapíše obsah do | ||
nového.)<br> | ||
<em>Program zkoušej v adresáři, který neobsahuje nic důležitého – pro | ||
Python není těžké přepsat soubor, ve kterém jsou důležité informace! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing </em>
. . . . . . . . X . | ||
</pre> | ||
<em>Jak na to? | ||
<ul> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not valid HTML: a block-level element (<ul>
) cannot be inside an inline element (<em>
).
Is this still relevant? Should I fix the conflict and so it can be merged in, even if the tasks aren't split up? |
I wouldn't merge it as is. But let's use it as a base for a split-up version. |
@encukou Could you please make an issue that describes what needs to be done instead? This is not mergable at all and the home projects are changing so it's not even cherry-pickable IMHO. |
Will do, but not this week. |
Well, I did a small write-up now: #277 This PR is linked there, so we can salvage whatever is useful. |
lessons
namaterials
.