Rozstřílení seminářové aplikace #60

Merged
zelvuska merged 19 commits from split into master 1 week ago
Owner

Přemýšlím, jestli tohle vůbec reviewovat, jestli nedává víc smysl v tom rovnou začít programovat dál a případně pak řešit „tohle jsem nemohl najít“ jednotlivě.

Speciálně tohle by asi chtělo nasadit dříve, než se noví webaři pustí do kódění (aby si nezvykli na starý řád věcí; je lepší řešit „tohle jsem nemohl najít“ než „tohle jsem nemohl najít, protože jsem to hledal tam, kde to vždycky bývalo“).

Přemýšlím, jestli tohle vůbec reviewovat, jestli nedává víc smysl v tom rovnou začít programovat dál a případně pak řešit „tohle jsem nemohl najít“ jednotlivě. Speciálně tohle by asi chtělo nasadit dříve, než se noví webaři pustí do kódění (aby si nezvykli na starý řád věcí; je lepší řešit „tohle jsem nemohl najít“ než „tohle jsem nemohl najít, protože jsem to hledal tam, kde to vždycky bývalo“).
zelvuska added 16 commits 3 months ago
zelvuska requested review from ledoian 1 month ago
Owner

Nechceme v tvorbě pojmenovat views_all.py jako views.py? Je tam ještě něco dalšího někde?

Nechceme v tvorbě pojmenovat `views_all.py` jako `views.py`? Je tam ještě něco dalšího někde?
ledoian reviewed 1 week ago
<p>
Na této stránce velmi intenzivně pracujeme.
Za dočasnou nedostupnost se omlouváme.
Zkuste přejít na <a href="/static">titulní stránku</a>
ledoian commented 1 week ago
Poster
Owner

ehh? FIXME?

ehh? FIXME?
zelvuska marked this conversation as resolved
Owner

Rozdělení various/views/pomocne.py a various/views/final.py mi nedává moc smysl, mám pocit, že v obou jsou dost random věci (vč. věcí které spíš patří do nějakých utils.py jako histogram()…) a tohle dělení mi přijde dost ortogonální na to, že ostatní jména souborů v various/views/ odpovídají použití té věci… Možná prostě sjednotit do various/views/other.py?

Rozdělení `various/views/pomocne.py` a `various/views/final.py` mi nedává moc smysl, mám pocit, že v obou jsou dost random věci (vč. věcí které spíš patří do nějakých `utils.py` jako `histogram()`…) a tohle dělení mi přijde dost ortogonální na to, že ostatní jména souborů v `various/views/` odpovídají použití té věci… Možná prostě sjednotit do `various/views/other.py`?
zelvuska added 1 commit 1 week ago
zelvuska added 1 commit 1 week ago
8d846647f6 Merge branch 'master' into split
zelvuska removed review request for ledoian 1 week ago
zelvuska added 1 commit 1 week ago
zelvuska merged commit 7108702e36 into master 1 week ago
zelvuska deleted branch split 1 week ago
The pull request has been merged as 7108702e36.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.