Automatyczne formatowanie kodu

[greg19]

Jako że w #11 miałem kilka uwag co do stylu kodu, to stwierdziłem że powinniśmy go jasno ustalić i zautomatyzować. Ten PR dodaje hook który przy każdym commicie formatuje wszystkie bloki wewnątrz plików markdown.

Moja propozycja formatowania znajdują się w pliku .clang-format. Z głównych decyzji, które aktualnie coś zmieniają to:

• Brak nowej linii między } a else.
• Limit na liczbę kolumn 120.

W razie innych opinii możemy dyskutować.

Aby rzeczywiście wszystko działo się automatycznie należy zainstalować narzędzie:

pip install pre-commit
pre-commit install

Wtedy przy każdym git commit uruchomi się formater na zmienionych plikach. Jeżeli coś mu się nie spodoba, to commit się nie powiedzie, a w plikach będą zmiany które możemy obejrzeć za pomocą git diff i zaakceptować za pomocą git add. Jak nam bardzo przeszkadza to można zrobić commita bez tego sprawdzenia za pomocą git commit --no-verify.

Aktualnie narzędzie ignoruje lekcje lekcje A2, A3 i A8, które specjalnie mają błędy. Jak ustalimy wspólny styl, to mogę je ręcznie dostosować.

[tonowak]

Dzięki za przygotowanie PRa! Nie wiedziałem, że istnieje hook, który formatuje wewnątrz markdown, dobre.

Przesyłam clang-format, z którego zazwyczaj korzystam, dopasowany pod mój styl. Jako, że styl kodu w kursie był modelowany według moich widzimisiów, to spodziewam się, że ta konfiguracja jest bliska aktualnemu kodowi. https://pastebin.com/SSBKEiFR Proponuję go użyć, lub jego drobną modyfikację.

Ja jestem za zmergowaniem, ale chciałbym byś najpierw spróbował pokazać i nauczyć @lduraj (np. przez meeta) jak korzystać z pre-commit.

Szerokość 120 znaków jak najbardziej OK, } else myślę że też, ale do tego jestem trochę mniej przekonany -- jeszcze nie zastanawiałem się które może być bardziej intuicyjne/prostsze dla początkujących, lub które jest bardziej używane w większych projektach.

[greg19]

Zaczerpnąłem kilka rzeczy z twojego configu. Możesz spojrzeć na diff między moją propozycją i twoim configiem. Jak coś nie pasuje to proszę o zgłoszenie.

Jak config wygląda dobrze, to rozumiem że zostaje się skontaktować z Lechem.

[tonowak]

To TabWidth: 8 wygląda podejrzanie, czy nie korzystamy z 4-spacjowych tabów? (nie pamiętam co to dokładnie robi).

Diff jest spory i musiałbym sporo googlować, więc nie widzę potrzeby bym go dokładniej przejrzał. Jeżeli mam się przyjrzeć jakiemuś konkretnemu fragmentowi, to daj znać.

[greg19]

Rzeczywiście to była pomyłka, poprawione. Nie mam nic na co chciałbym zwrócić uwagę w diffie.