Hw12 calendar #12
Hw12 calendar #12
Conversation
There was a problem hiding this comment.
Здравствуйте. Спасибо за выполненное задание. Хорошая работа
Итого: 10/10
Нотация:
👍 - Классное, не обычное, оригинальное решение
👀 - Стоит обратить внимание. Рекомендация, которая не влияет на успешность выполнения задания
🔥 - Критичное замечание, которое влияет на сдачу задания
| @@ -0,0 +1,15 @@ | |||
| run: | |||
There was a problem hiding this comment.
👀
Зачем вы добавили новую конфигурацию линтера, если у вас она есть на каталог выше?
| @@ -0,0 +1,15 @@ | |||
| run: | |||
There was a problem hiding this comment.
👀
Зачем вы правите конфигурацию линтера, если она у вас есть выше на директорию?
|
|
||
| GIT_HASH := $(shell git log --format="%h" -n 1) | ||
| LDFLAGS := -X main.release="develop" -X main.buildDate=$(shell date -u +%Y-%m-%dT%H:%M:%S) -X main.gitHash=$(GIT_HASH) | ||
| DB_USER := "root" |
There was a problem hiding this comment.
👀
Надеюсь в продакшен конфигурации вы так не делаете)))
| cancel() | ||
| os.Exit(1) //nolint:gocritic | ||
| Logger.Error("failed to start http server: " + err.Error()) | ||
| os.Exit(1) |
There was a problem hiding this comment.
👀
Не рекомендую использовать os.Exit(...) в своих промышленных программах, так как при таком завершении программы никогда не выполнятся отложенные операции
|
|
||
| Logger, err = logger.New(config.Logger.Level, config.Logger.File) | ||
| if err != nil { | ||
| fmt.Println(err.Error()) |
| @@ -0,0 +1,248 @@ | |||
| package sqlstorage | |||
|
|
|||
| //import ( | |||
There was a problem hiding this comment.
👀
Если не нужны тесты, то лучше их не включать в ПР
Домашнее задание №12 «Заготовка сервиса Календарь»
Критерии оценки
на пакеты по определенной логике) - до 2 баллов
Реализовано хранилище:
Зачёт от 7 баллов