Skip to content
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

Beginners updates from brno #38

Closed
wants to merge 6 commits into from

Conversation

tomjelinek
Copy link
Contributor

  • fix typos and similar mistakes
  • add missing comas in sentences
  • add a placeholder 'lesson' used in Brno course
  • do not recommend VScode - there has been a discussion and a decision about this in Brno, but nobody did the actual job of modifying the text

Copy link
Member

@hroncok hroncok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Změny v čárkách ve větách a jiné překlepy vpadají velmi dobře, díky. Jednou věcí si nejsem jistý.

Vypiš si teď obsah aktuálního adresáře pomocí `ls` nebo `dir`.
Jeden z vypsaných adresářů bude `zkouska`.

Když je adresář vytvořený, můžeš do něj přejít, podobně jako jsi před chvílí
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nejsem si jistý čárkou před podobně jako. Zkoušel jsem si o tom něco přečíst, ale nejsem z toho moc moudřejší. Srovnáváme tady, nebo slučujeme? Podle mě tady srovnáváme, a čárka tam být tedy nemá. Ale ruku do ohně za to nedám.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Taky jsem si nebyl moc jistý. Vrátím to zpátky na verzi bez čárky.

@@ -289,10 +289,10 @@ Pozor na to, že když nenapíšeš závorky, funkce se nezavolá!
Výraz `len(s)` je *volání funkce*, ale `len` bez závorek označuje
*funkci samotnou*.

Výsledek `len(s)` je číslo; `len` je funkce.
Výsledek `len(s)` je číslo; výsledek `len` je funkce.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tahle změna mi přijde méně srozumitelná než originál.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Co tohle: Výsledek výrazu len(s) je číslo; výsledek výrazu len je funkce len.

Šlo mi o to, aby tam nebylo jen 'len je funkce'. To je napsáno hned nad tím.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...výsledek výrazu len je samotná funkce len.

@tomjelinek tomjelinek requested a review from hroncok March 1, 2024 07:59
Copy link
Member

@hroncok hroncok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Followup změny jsou OK, na review obsahových změn ale nemám kapacitu.

@tomjelinek tomjelinek force-pushed the beginners-updates-from-brno branch from 1d2b154 to d851d83 Compare March 2, 2024 17:58
@encukou
Copy link
Member

encukou commented Apr 27, 2024

  • add a placeholder lesson: Tohle dává smysl pro brněnský kurz, ale pro samouky nebo ostatní kurzy moc ne.
  • fix typos and do minor improvements: super, díky!
  • fix order of mkdir and cd: Taky super, díky!
  • do not recommend VScode: Tohle si zase každý kurz musí určit podle sebe

Ty dvě změny začlením (#41); zbytek bude lepší mít ve větvi pro Brno.

@encukou encukou closed this Apr 27, 2024
@tomjelinek tomjelinek deleted the beginners-updates-from-brno branch May 5, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants