search

pupenasan / pacframework-tools

MIT License
1 stars 0 forks source link

Introduce linter #4

Open vozemer opened 1 year ago

vozemer commented 1 year ago

Rationale for linter

Linter can analyze the code and find many sorts of bugs and deoptimizations. It enforces good practices distilled by thousands of hours spent by developers.

Rationale for code style

Enforcing code style helps with code consistency so during the PR review process we can skip formatting concerns :) Also, it helps to remove cognitive load when you see different code formatting in the same project.

vozemer commented 1 year ago

Addressed here: https://github.com/pupenasan/pacframework-tools/pull/6

pupenasan commented 1 year ago

дякую. А як ним користуватися у VS code? Запускати скрипт lint?

vozemer commented 1 year ago

@pupenasan краще поставити https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint Тоді буде інтеграція між VSCode та ESLint, індикація помилок (error, warn) та інше. Також можна повісити запуск лінтера на зберігання файлу, тоді він автоматично буде перевіряти і виправляти.