search

polina-nn / news

This is an educational project. The task is here - https://coda.io/@metalamp/education/4-15
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Оформление. #2

Closed pavelzarubin closed 1 year ago

pavelzarubin commented 1 year ago

Я не знаю какой у тебя форматтер, но ormolu файлы меняет. Есть варнинги от hlint. Есть лишние комментарии.

pavelzarubin commented 1 year ago

Если пользуешься VSCode поставь Code Spell Checker расширение, имена функций чтобы назывались правильно.

pavelzarubin commented 1 year ago

https://github.com/polina-nn/news/blob/c6c0b6d15155c6c7a0bf7b33fa6046d6c6131182/src/Types/DataTypes.hs#L218-L223

в Haskell мы используем camelCase

polina-nn commented 1 year ago

https://github.com/polina-nn/news/pull/new/2_omoru_camelCase_hlint

pavelzarubin commented 1 year ago

У меня ormolu до сих пор вносит справления? у тебя какая версия используется. Я попробовал несколько, 0.1.4.1, 0.3_ и последнюю, все три вносят изменения

polina-nn commented 1 year ago

Привет, я еще та курица. Я умудрилась после ormolu, прогнать через hindent.😂 Я сделала исправления по ишью про базу данных, завтра выложу и перед этим прогоню только через ormolu. Надеюсь наши варианты будут теперь совпадать. Я на дежурстве, не могу глянуть версию ormolu….

 

Вторник, 27 декабря 2022, 9:35 +03:00 от pavelzarubin @.***>:     У меня ormolu до сих пор вносит справления? у тебя какая версия используется. Я попробовал несколько, 0.1.4.1, 0.3_ и последнюю, все три вносят изменения — Reply to this email directly, view it on GitHub , or unsubscribe . You are receiving this because you modified the open/close state. Message ID: <polina-nn/news/issues/2/1365650007 @ github . com>    

Полина Морозова