-
Notifications
You must be signed in to change notification settings - Fork 118
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
Vyřešit flake8 #587
Comments
Koukám, že konkrétní postup je jen u Atomu. U Kate a Gedit to není vůbec a u Ostatní je to jen obecně popsáno bez návodu. Podle mě je jediný problém v tom, že si účastnice instalují flake8 do virtuálního prostředí, kde jej editor nenajde. Takže to můžeme:
Já bych byl za zachování a třeba i oddělení do samostatné bonusové části, ale snad všichni to v nějaké podobě používáme, tak bych jim to také rád ukázal. |
cc @zuzejk |
Tady bohužel nevím, poslední běh jsem moc nekoučoval. Ale v materiálech teď vidím VSCode a flake8 se tam neřeší vůbec. Ani nevím, jestli si ten VSCode vůbec nainstalují i s tou Python extension. Takže jak je to odzkoušeno? :) |
VSCode otravuje, že doporučuje nějaké svoje rozšíření doinstalovat. Někdo na to klikne, někdo ne. Můj názor je pořád ten, že úplně na začátku je spousta jiných problémů. Nechci přidávat ještě kontrolu stylu. |
Mně taky přijde, že to generuje jen problémy. |
Jsem raději, když začátečníky žádný linting nerozptyluje. Na druhou stranu někdy se dost zaseknou, když jim někde chybí dvojtečka, oni nevědí, co se děje (neumí dešifrovat SyntaxError), tam by linting pomohl, ale takhle je to aspoň naučí řešit problémy místo tabulator-driven developmentu. Btw. nebyla tato diskuze už někde? Nebo je zde správné místo pro ni? |
Tohle je správné místo. (Jestli štítek |
Ze Slacku:
The text was updated successfully, but these errors were encountered: