Pavel "LEdoian" Turinsky
3 years ago
27 changed files with 358 additions and 65 deletions
@ -0,0 +1,173 @@ |
|||
# Postup zkrášlení kódu M&Mího webu |
|||
|
|||
## Obecně o webu |
|||
|
|||
- Python, Django, spousta nějakých rozšíření, frontend HTML + CSS + trocha JS |
|||
- Velké břímě historie, kterou nejspíš nechceme zahodit |
|||
- Změny v M&M někdy dost zamotají potřebný kód (tituly) |
|||
- Občas je potřeba dělat opravy rychle |
|||
|
|||
## Aktuální stav |
|||
|
|||
- Zběsile zbastlený kód |
|||
- „Co je to ‚single responsibility principle‘?“ ☺ |
|||
- Dost netriviální množství objektů a jejich vazeb |
|||
- Dost možná do velké míry inherentní složitost |
|||
- Webaři aktuálně relativně zběhlí v programování (ale je potřeba myslet na to, že to tak být nemusí) |
|||
- Webaři stárnou (Jethro, Kristý, Anet) a mizí (Pavel, Káťa), je potřeba web připravit na předání |
|||
- Kód je rozdělený mezi orgy a jeden „nerozumí“ tomu, co druhý napsal (musí to vyčíst z kódu, nezná souvislosti, …) |
|||
- Noví orgové se aktuálně musí ptát, což je jim nepříjemné a nutí je umět formulovat dotazy |
|||
|
|||
### Invarianty |
|||
|
|||
- Není to práce, ale zábava-ish → libovolný proces nesmí být (moc) na obtíž. |
|||
- I malé nepohodlí je potřeba vyvážit relativně velkým přínosem |
|||
- nebo dostatečně zřejmou vidinou budoucího pohodlí / minimalizace nepohodlí |
|||
- Nejsme programátoři, spíš jsme bastlíři kódu (kteří znají rozumnou podmnožinu syntaxe Pythonu) |
|||
- Zvlášť noví orgové |
|||
- Nechceme cílit na mega-profi kód, je to nedosažitelný cíl |
|||
- Nejspíš to do nějaké míry ubastlené bude pořád, ta míra závisí na zkušenosti aktuálních webařů |
|||
- Nástroje nás nesmí moc mást. |
|||
- Děláme to zadarmo jeden večer v týdnu |
|||
- Vývoj jde pomalu, často pomaleji než vývoj knihoven |
|||
- → kód se rozbíjí i sám |
|||
- Běží to na Gimlim |
|||
- Debian (old)stable → nemůžeme používat moc nové featury Pythonu |
|||
- Aktuálně Python 3.7.3 |
|||
- Webaři jsou náhodně vzniklá skupina lidí. |
|||
- Různé nástroje, různé operační systémy |
|||
- Nechceme vynucovat konkrétní metody, multiplatformní nástroje asi požadovat můžeme |
|||
- Na serveru může běžet cokoliv, co tam jde rozběhnout |
|||
- Kód by neměl být moc složitý / matoucí / kompaktní (?) |
|||
- případně fakt hodně okomentovaný |
|||
|
|||
## O co se snažíme |
|||
|
|||
- Zpřístupnit vývoj novým webařům |
|||
- Umožnit chápání i cizího kódu co nejjednodušeji |
|||
- Nevzít si s sebou implementační tajemství ~~do hrobu~~ pryč z M&Mka |
|||
|
|||
```graphviz |
|||
digraph "Závislosti věcí" { |
|||
ss -> sdil -> doku -> cs; |
|||
ss -> cit ->ref -> cs; |
|||
ref -> nrt -> tst -> cs; |
|||
sdil -> cr -> cs |
|||
ss -> nrt; |
|||
ss[label="Současný stav",shape=box]; |
|||
cit[label="čitelný kód"]; |
|||
cs[label="Cílový stav",shape=box]; |
|||
ref[label="refactoring"]; |
|||
nrt[label="nerozbít to"]; |
|||
tst[label="testy"]; |
|||
doku[label="dokumentace (vývojářská)"]; |
|||
sdil[label="chápání kódu ostatních / stávajícího"]; |
|||
cr[label="code review"]; |
|||
nrt,sdil,cit[shape=hexagon]; |
|||
tst,doku,cr[color=blue]; |
|||
} |
|||
``` |
|||
|
|||
## Code review |
|||
|
|||
Aktuálně: Pavel občas z rozmaru čte diffy; párkrát jsme zkoušeli [programovat v páru](https://mam.mff.cuni.cz/wiki/Web/Tipy/PairProgramming), je to relativně časově náročné. |
|||
|
|||
- Nevynucovat |
|||
- Primární motivace je umožnit nějak vidět změny a případně k nim dávat komentáře, jak stylu „tohle mi není jasné“, tak i „tohle by chtělo přepsat“. |
|||
- Chceme hlavně vytvořit příležitost ke čtení cizího kódu a seznamování se s ním (i v zájmu zaučování nových webařů) |
|||
|
|||
### Názory |
|||
|
|||
- spíš post-hoc |
|||
- Možná code-review toho, co jde na produkci |
|||
- Chceme umět komentovat konkrétní řádky kódu |
|||
|
|||
- Gitea/gitlab? |
|||
- Klikátko a barevný řádky a vyhledávání jsou fajn (gitlab, gitea) |
|||
- Jethro: je fajn umět skočit na definici |
|||
- Kombinace s CI |
|||
- Pokud bude gitea v něčem nedostatečná, tak hrozí, že bude potřeba migrovat znovu, což je nepraktické |
|||
- Gitlab je asi častější než gitea → je lepší názor si na to zvyknout |
|||
|
|||
__Závěr:__ Zkusíme to hodit do GitLabu (nejspíš veřejného\*), zavedeme pull-requesty do `master` větve, náhodně se budeme přiřazovat a používat ho nějak intuitivně. |
|||
|
|||
\*: Ve fakultním neumíme přidávat další uživatele, v KAMím zas možná není CI. |
|||
|
|||
## Testy |
|||
|
|||
Aktuálně: pár testů na dohromady řádově 4 funkce, drobný pokus o TDD, jenž narazil na úskalí reálného světa. Lokální spuštění testů trvá relativně dlouho, nejspíš kvůli spoustě migrací. |
|||
|
|||
- Potřebujeme ověřit, že to funguje a že to _pořád_ funguje |
|||
|
|||
- CI? Coverage? |
|||
|
|||
### Názory |
|||
|
|||
- PyTest je fajn |
|||
- Testovat frontend? |
|||
- Jethro: při nasazení by se mohly dělat screenshoty celých vybraných stránek přes Selenium (nebo obdobné) a hlásit rozdíly |
|||
|
|||
__Závěr:__ Backend testujeme PyTestem (`./manage.py test`), u frontendu výhledově zkusíme ty vizuální diffy a pak se uvidí, CI podle webového klikátka na code review. Bylo by dobré mít rozumná testdata, ať se nemusí mockovat moc věcí. |
|||
|
|||
## Formát kódu |
|||
|
|||
Aktuálně: Jakýsi coding style zhruba existuje, není popsaný, šíří se lidovou slovesností. |
|||
|
|||
- Nesmí být striktně vynucovaný |
|||
- Musel by být hodně nastavitelný |
|||
- Nechceme mít kód plný `#NOQA: WTF42` |
|||
- Nejspíš vždycky bude mít false positives (`seminar.utils.roman_numerals`) i false negatives (`seminar.models.tvorba.Cislo.posli_cislo_mailem`) |
|||
- Možná dobrý sluha, ale určitě špatný pán (also: špatná zkušenost ☺) |
|||
- __Důsledek:__ Hrozí, že těch falešných varování bude moc, čímž to ztratí smysl úplně |
|||
- Potenciálně by šlo aplikovat jen lokálně na změny? |
|||
|
|||
### Názory |
|||
|
|||
- P: nemyslím si, že zvládneme mít „průhledný kód“ (dostatečně konzistentní kód, aby ho člověk přestal vnímat a spíš viděl myšlenky). |
|||
|
|||
- P: Kecadla do kódu trochu zavání větším peklem než užitkem (takže black a flake8 jsou ze hry); isort možná dává smysl; je otázka, jestli použít mypy, ale typové značky v kódu spíš chceme (zvlášť u věcí, které nejsou prostě django view – typicky utils.) |
|||
- J: Divné (=Pavlovo) groupování importů je spíš matoucí, spíš moc nepomáhalo |
|||
|
|||
- P (doplněno zpětně): pydocstyle vynucuje PEP-257, který se možná tluče se sphinxem… |
|||
- P (též zpětně): Možná by se mohl dát použít pylint, tomu jde aspoň vysvětlit coding style a nerazí tupě PEP-8, ale nastavovat ho je asi větší porod, než jak moc pomůže… |
|||
|
|||
__Závěr:__ Kecadla na formát spíš nechceme; isort by mohl být fajn, ale bylo by dobré mít rozdělené bloky „náš kód“, „standardní knihovna“ a „Django a další balíčky z PyPA“; u našeho kódu (utils, obecně ne-Django věci) chceme držet záměr (na úrovni dohody) psát signatury funkcí a v případě jejich přítomnosti se nechat varovat při porušení signatury. |
|||
|
|||
- P (večerní prokrastinace): Bandit vypadá, že hlásí jen strašně obvious věci by default, nepřijde mi jako moc úžasné vylepšení. Do CI možná dobrý |
|||
|
|||
## Dokumentace |
|||
|
|||
Aktuálně: něco málo je na wiki (`/Web/`), občas má nějaká funkce docstring, obecně je toho málo |
|||
|
|||
### Požadavky |
|||
|
|||
- Jedno autoritativní místo a dá se najít |
|||
- Dostatečně „blízko“ kódu |
|||
- nastavit mindset na psaní dokumentace „rovnou“? |
|||
- Umožnit i obecný text, ne jen komentáře kódu (modulů, funkcí) |
|||
|
|||
|
|||
### Bonusy |
|||
|
|||
- Zajišťování konzistence s uživatelskou dokumentací |
|||
- aktuálně wiki |
|||
- Podpora i dalších jazyků (Vue, Javascript, CSS, možná django-templates) |
|||
|
|||
### Názory |
|||
|
|||
- Jde to dělat sphinxem |
|||
- Stínovlas by mohl vědět v CZ.NICu se sphinx jede. Případně zkusit zjistit, co umí jejich Akademie |
|||
- Free-textová dokumentace (architektura ap.) má být nezávislá na dokumentaci konkrétního kódu, ale je fajn, když se to pak spojí dohromady, jde-li to. |
|||
- Lepší, když se obojí dá dělat stejným nástrojem |
|||
- Káťa má někdy pocit, že tráví spoustu času tím, že hledá který soubor vůbec upravit; to má v naší dokumentaci být. |
|||
|
|||
__Závěr:__ Zkusíme použít sphinx (z nedostatku vlastních zkušeností a kvůli jeho popularitě), když se nám to nebude líbit, tak budeme řešit dál. Bylo by fajn najít nějaký workshop, bude to rychlejší nalejvárna než číst dokumentaci. V krátkodobém výhledu stáhneme vývojovou dokumentaci z webu do repozitáře a zprovozníme někde automatické buildy (aby se dala číst jako člověk a ne jako stroj). |
|||
|
|||
## Uživatelská dokumentace |
|||
|
|||
- K: Dělat ji ve spolupráci s těma uživatelema |
|||
- P: Dávný Hedgedoc může případně sloužit jako základ osnovy |
|||
- K: Dost možná nevíme, co bolí víc a co míň |
|||
- Jethro: Musí být na wiki, jinak tím zmateme oržstvo |
|||
|
|||
__Závěr:__ Dokumentace bude na wiki (<https://mam.mff.cuni.cz/wiki/Web_user/Dokumentace>, výhledově možná ve stejné složce v dalších souborech), bude vznikat podle našich pocitů a dotazů od ostatních; výhledově bude schůzka na ukázání featur nového webu a tam se dosbírají náměty na to, co sepsat. |
|||
@ -0,0 +1,18 @@ |
|||
# Generated by Django 2.2.24 on 2021-12-05 23:09 |
|||
|
|||
from django.db import migrations, models |
|||
|
|||
|
|||
class Migration(migrations.Migration): |
|||
|
|||
dependencies = [ |
|||
('korektury', '0017_auto_20190610_2358'), |
|||
] |
|||
|
|||
operations = [ |
|||
migrations.AddField( |
|||
model_name='korekturovanepdf', |
|||
name='poslat_mail', |
|||
field=models.BooleanField(default=True, help_text='Určuje, zda se má o nově nahraném PDF poslat e-mail do mam-org. Při upravování existujícího souboru už nemá žádný vliv.', verbose_name='Poslat mail o novém PDF'), |
|||
), |
|||
] |
|||
@ -1,19 +1,16 @@ |
|||
{% extends "base.html" %} |
|||
{% extends "error_base.html" %} |
|||
|
|||
{% load staticfiles %} |
|||
|
|||
{% block content %} |
|||
<h2> |
|||
{% block nadpis1a %}{% block nadpis1b %} |
|||
{% block errorheading %} |
|||
O-jo-jo-jo-joj |
|||
{% endblock %}{% endblock %} |
|||
</h2> |
|||
{% endblock %} |
|||
|
|||
<p> |
|||
{% block errortext %} |
|||
Chybička se vloudila. |
|||
Zkuste přejít na <a href="/">titulní stránku</a> |
|||
nebo se podívat na <a href="{% url 'seminar_aktualni_zadani' %}">aktuální zadání</a>. |
|||
</p> |
|||
{% endblock %} |
|||
|
|||
{% block errorimage %} |
|||
<img src="{% static '500.png' %}"> |
|||
{% endblock %} |
|||
|
|||
|
|||
@ -1,19 +1,16 @@ |
|||
{% extends "base.html" %} |
|||
{% extends "error_base.html" %} |
|||
|
|||
{% load staticfiles %} |
|||
|
|||
{% block content %} |
|||
<h2> |
|||
{% block nadpis1a %}{% block nadpis1b %} |
|||
{% block errorheading %} |
|||
Vrrrrrrrrr |
|||
{% endblock %}{% endblock %} |
|||
</h2> |
|||
{% endblock %} |
|||
|
|||
<p> |
|||
{% block errortext %} |
|||
Tady pravděpodobně nemáte co dělat. |
|||
Zkuste přejít na <a href="/">titulní stránku</a> |
|||
nebo se podívat na <a href="{% url 'seminar_aktualni_zadani' %}">aktuální zadání</a>. |
|||
</p> |
|||
{% endblock %} |
|||
|
|||
{% block errorimage %} |
|||
<img src="{% static '500.png' %}"> |
|||
{% endblock %} |
|||
|
|||
|
|||
@ -1,19 +1,16 @@ |
|||
{% extends "base.html" %} |
|||
{% extends "error_base.html" %} |
|||
|
|||
{% load staticfiles %} |
|||
|
|||
{% block content %} |
|||
<h2> |
|||
{% block nadpis1a %}{% block nadpis1b %} |
|||
Požadovaná stránka nenalezena |
|||
{% endblock %}{% endblock%} |
|||
</h2> |
|||
{% block errorheading %} |
|||
Požadovaná stránka nenalezena |
|||
{% endblock %} |
|||
|
|||
{% block errortext %} |
|||
Tuto stránku jsme na našem serveru nenalezli. |
|||
{% endblock %} |
|||
|
|||
<p> |
|||
Tuto stránku jsme na našem serveru nenalezli. |
|||
Zkuste přejít na <a href="/">titulní stránku</a> |
|||
nebo se podívat na <a href="{% url 'seminar_aktualni_zadani' %}">aktuální zadání</a>. |
|||
</p> |
|||
<img src="{% static '404.png' %}"> |
|||
{% block errorimage %} |
|||
<img src="{% static '404.png' %}"> |
|||
{% endblock %} |
|||
|
|||
|
|||
@ -1,20 +1,16 @@ |
|||
{% extends "base.html" %} |
|||
{% extends "error_base.html" %} |
|||
|
|||
{% load staticfiles %} |
|||
|
|||
{% block content %} |
|||
<h2> |
|||
{% block nadpis1a %}{% block nadpis1b %} |
|||
{% block errorheading %} |
|||
O-jo-jo-jo-joj |
|||
{% endblock %}{% endblock %} |
|||
</h2> |
|||
{% endblock %} |
|||
|
|||
<p> |
|||
{% block errortext %} |
|||
Chybička se vloudila. |
|||
Zkuste přejít na <a href="/">titulní stránku</a> |
|||
{# Při pádu webu nechceme, aby se rozbily odkazy, nevěříme URLconfu. #} |
|||
nebo se podívat na <a href="/aktualni/zadani/">aktuální zadání</a>. |
|||
</p> |
|||
{% endblock %} |
|||
|
|||
{% block errorimage %} |
|||
<img src="{% static '500.png' %}"> |
|||
{% endblock %} |
|||
|
|||
|
|||
@ -0,0 +1,19 @@ |
|||
{% extends "base.html" %} |
|||
|
|||
{% block content %} |
|||
<h2> |
|||
{% block nadpis1a %}{% block nadpis1b %} |
|||
{% block errorheading %} |
|||
{% endblock %} |
|||
{% endblock %}{% endblock%} |
|||
</h2> |
|||
|
|||
<p> |
|||
{% block errortext %} |
|||
{% endblock %} |
|||
Zkuste přejít na <a href="/">titulní stránku</a>. |
|||
</p> |
|||
{% block errorimage %} |
|||
{% endblock %} |
|||
{% endblock %} |
|||
|
|||
@ -0,0 +1,24 @@ |
|||
# Generated by Django 2.2.24 on 2021-12-13 22:06 |
|||
|
|||
from django.db import migrations, models |
|||
import django.db.models.deletion |
|||
|
|||
|
|||
class Migration(migrations.Migration): |
|||
|
|||
dependencies = [ |
|||
('seminar', '0100_auto_20211129_2354'), |
|||
] |
|||
|
|||
operations = [ |
|||
migrations.AlterField( |
|||
model_name='nastaveni', |
|||
name='aktualni_cislo', |
|||
field=models.ForeignKey(on_delete=django.db.models.deletion.PROTECT, to='seminar.Cislo', verbose_name='Aktuální číslo'), |
|||
), |
|||
migrations.AlterField( |
|||
model_name='soustredeni', |
|||
name='typ', |
|||
field=models.CharField(choices=[('jarni', 'Jarní soustředění'), ('podzimni', 'Podzimní soustředění'), ('vikend', 'Víkendový sraz'), ('vylet', 'Výlet')], default='podzimni', max_length=16, verbose_name='typ akce'), |
|||
), |
|||
] |
|||
Loading…
Reference in new issue