-
Notifications
You must be signed in to change notification settings - Fork 119
Pyknihovny 3 #79
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
Pyknihovny 3 #79
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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. |
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.
typo traadičně
|
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nástroje/příkazu pip (jinak by to bylo pomocí pipu a skloňovat to moc nechceme)
lessons/intro/distribution/index.md
Outdated
| 1 directory, 5 files | ||
| ``` | ||
|
|
||
| Soubor `__init__.py` jednak značí, že adresář `isholiday` je Pythonní modul, |
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Najednou se mluví o __main__.py, nemá to být __init__.py?
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.
Má; thinko
| entry_points={ | ||
| 'console_scripts': [ | ||
| 'executable_name = isholiday.holidays:main', | ||
| 'isholiday_demo = isholiday.holidays:main', |
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.
👍 tohle tam lidi nechávali
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
opět malé p (asi třeba zkontrolovat celý dokument)
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 <[email protected]> Co-authored-by: Lumír 'Frenzy' Balhar <[email protected]>
Příprava na 3. lekci kurzu "Python a jeho knihovny".
Odebrání úkolu, menší změny v obsahu, přidání do osnovy.