Conversation
|
Chceme takhle odkazovat na úkol? Nechal bych zadání úkolu jen na MI-PYTu, kde odkazujeme na materiály. Odkaz an úkol z materiálů je z hlediska MI-PYTu už navíc. |
lessons/intro/distribution/index.md
Outdated
| Zatím jsme tvořili programy v Pythonu tak nějak na divoko, tedy v jednom nebo | ||
| více souborech bez nějakého zvláštního řádu. Na dnešním cvičení se podíváme na | ||
| to, jak tvořit redistribuovatelné moduly, které jdou instalovat pomocí pipu. | ||
| více souborech bez nějakého zvláštního řádu. Dnes se podíváme na |
There was a problem hiding this comment.
Pokud má text fungovat i pro čtenáře zvenčí, dal bych místo "dnes" "v této lekci".
lessons/intro/distribution/index.md
Outdated
|
|
||
| Jak vidíte, `setuptools` si stěžuje, že náš projekt nemá `README`. | ||
| Jak vidíte, `setuptools` si stěžuje, že náš projekt nemá `README` – soubor, | ||
| do kterého se traadičně píšou základní informace o projektu. |
|
Z hlediska naucse, nebo aspoň aktuálního kurzu, je odkaz na úkol užitečný. A i pro MI-PYT mi přijde fajn tam ten odkaz mít. |
lessons/intro/distribution/index.md
Outdated
| ... | ||
| ``` | ||
|
|
||
| Hotový balíček pak můžete nainstalovat pomocí `pip`. |
There was a problem hiding this comment.
nástroje/příkazu pip (jinak by to bylo pomocí pipu a skloňovat to moc nechceme)
lessons/intro/distribution/index.md
Outdated
| @@ -196,9 +222,9 @@ V takovém případě uděláme modul ve formě složky. V našem případě sou | |||
| ``` | |||
|
|
|||
| Soubor `__init__.py` jednak značí, že adresář `isholiday` je Pythonní modul, | |||
There was a problem hiding this comment.
pythonní malé p (není změněno, ale tady jsem si všiml)
lessons/intro/distribution/index.md
Outdated
| pouze importovat veřejné rozhraní, tedy to, co budou z vašeho modulu importovat | ||
| jeho uživatelé. | ||
|
|
||
| Přesuňte tedy obsah `__main__.py` do `holidays.py`, a do `__main__.py` |
There was a problem hiding this comment.
Najednou se mluví o __main__.py, nemá to být __init__.py?
| entry_points={ | ||
| 'console_scripts': [ | ||
| 'executable_name = isholiday.holidays:main', | ||
| 'isholiday_demo = isholiday.holidays:main', |
lessons/intro/distribution/index.md
Outdated
|
|
||
| ### Soubor requirements.txt | ||
|
|
||
| Kromě závislostí v `setup.py` se u Pythonních projektů často setkáme se souborem |
There was a problem hiding this comment.
opět malé p (asi třeba zkontrolovat celý dokument)
There was a problem hiding this comment.
Zkontroloval jsem to pomocí grep Pythonn
lessons/intro/distribution/index.md
Outdated
| [================================] 8379/8379 - 00:00:02 | ||
| ``` | ||
|
|
||
| Registrace se zdaří jen pokud jméno projektu již není zabrané. |
There was a problem hiding this comment.
Opět nesouvisející chyba. Za "zdaří" patří čárka.
lessons/intro/distribution/index.md
Outdated
| než ta, co už na PyPI je. Nejde tedy jednou nahraný modul přepsat. | ||
|
|
||
| Pro nahrání na opravdovou PyPI stačí vynechat `-r pypitest`. | ||
| Zabírat jména na PyPI jen tak není hezké vůči ostatním Pythonistům; registrujte |
There was a problem hiding this comment.
Možná bych dal "na opravdové PyPI", na testovací ať si každý registruje, co chce.
hroncok
left a comment
There was a problem hiding this comment.
Přidal jsem komentáře mimo review. Odkaz na úkol tedy nechme.
* Fix: title attribute on img tag (pyvec#49) * Remove course definition * WIP pydata databases * WIP pydata databases * WIP pydata databases * vraceni zmeny v nesouvisejici lekci * oprava databases/info.yml * WIP * WIP * WIP * WIP * WIP * WIP * WIP * movies.sqlite * WIP * WIP * WIP * MVP Co-authored-by: Jan Pipek <jan.pipek@gmail.com> Co-authored-by: Lumír 'Frenzy' Balhar <frenzy.madness@gmail.com>
Příprava na 3. lekci kurzu "Python a jeho knihovny".
Odebrání úkolu, menší změny v obsahu, přidání do osnovy.