-
Notifications
You must be signed in to change notification settings - Fork 1
Добавляет поддержку Dhall для управления конфигами #32
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
base: master
Are you sure you want to change the base?
Conversation
|
Спасибо, но сейчас времени нет. Думаю на выходных посмотреть, dhall нам в будущем пригодится. |
|
Круто, т.е. даже не надо явно парсить dhall-файл? Он сам читает в соответствии с определениями хаскельных типов? @Znack я помню, ты кажется говорил, что мы специально конфиг не кешируем, чтобы правки конфига вступали в силу без перезапуска приложения. Это актуально? Честно говоря, мне это не кажется безопасной схемой :). Но если актуально, то надо это иметь в виду при использовании dhall. |
в итоге мы этим не пользуемся и скорее всего не воспользуемся никогда :) так что я бы не заморачивался |
| runAllMigrations :: IO () | ||
| runAllMigrations = do | ||
| conf <- C.retrieveConfig | ||
| conf <- C.loadConfig "./config/dev.dhall" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
это точно надо поправить, нехорошо что везде путь таскать надо. В файле конфига его константой сохранить хотя бы
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это я оставил в туду в описании PR. Если у кого будет возможность то можно исправить.
#21
Пока сделал первую версию для загрузки конфигов.
В планах
Если есть какие-то предложения, то давайте тут обсуждать.