Skip to content

Conversation

@MangoosteMA
Copy link
Collaborator

No description provided.

@grphil
Copy link
Collaborator

grphil commented Apr 16, 2025

Кажется этот PR надо отложить. У нас есть много мест, где в целом непонятно как фиксить (например logger.err возвращает error, и он то юзается, то нет)

@MangoosteMA
Copy link
Collaborator Author

Если результат logger.Error("error") не используется, то есть два способа этого избежать:

_ = logger.Error("error")

или

logger.Error("error") //nolint

Комментарий //nolint можно оставить и на предыдущей строке, но мне кажется, что гармоничнее он смотрится на той же строке, к которой относится. Также можно писать комментарий с пробелом // nolint :)

На мой взгляд оба способа норм

@grphil
Copy link
Collaborator

grphil commented Apr 17, 2025

Ну в любом случае давай сначала хоть как-то заставим все работать, а потом линтером и всякими правильными названиями займемся

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.

3 participants