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

[SOK-10] pre-commit #6

Merged
merged 7 commits into from
Sep 27, 2024
Merged

[SOK-10] pre-commit #6

merged 7 commits into from
Sep 27, 2024

Conversation

Timur233
Copy link
Owner

### Какую задачу решаем

Настроил precommit хук, на тесты и линтер

Скриншоты/видяшка (если есть)

TBD (если есть)

Позже можно будет настроить линтер если будет желание

Copy link
Collaborator

@k0ndratov k0ndratov left a comment

Choose a reason for hiding this comment

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

Круто, спасибо что обратил на это внимание.

Попытался проверить форматирование prettier. Выглядит так, что husky проводит проверку, но никтогда не падает, что странно, ведь команда yarn format приводит к очевидным изменениям.

Выглядит так, что требует внимания со стороны prettier`а.

Тестировал вот так:
https://github.com/Timur233/falcon-tanks/commits/test-husky/

@Timur233 Timur233 requested a review from k0ndratov September 27, 2024 07:36
@Timur233 Timur233 merged commit d63382f into develop Sep 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants