Skip to content

Conversation

@kelizarov
Copy link
Collaborator

#21

Пока сделал первую версию для загрузки конфигов.

В планах

  • парсить пусть к файлу через ENV, если нет пытаться загрузить дефолтный в корне
  • привести нейминг модулей в один стиль

Если есть какие-то предложения, то давайте тут обсуждать.

@antonkalinin-ml
Copy link
Contributor

Спасибо, но сейчас времени нет. Думаю на выходных посмотреть, dhall нам в будущем пригодится.

@antonkalinin-ml
Copy link
Contributor

Круто, т.е. даже не надо явно парсить dhall-файл? Он сам читает в соответствии с определениями хаскельных типов?

@Znack я помню, ты кажется говорил, что мы специально конфиг не кешируем, чтобы правки конфига вступали в силу без перезапуска приложения. Это актуально? Честно говоря, мне это не кажется безопасной схемой :). Но если актуально, то надо это иметь в виду при использовании dhall.

@Znack
Copy link
Contributor

Znack commented Nov 22, 2021

я помню, ты кажется говорил, что мы специально конфиг не кешируем, чтобы правки конфига вступали в силу без перезапуска приложения. Это актуально?

в итоге мы этим не пользуемся и скорее всего не воспользуемся никогда :) так что я бы не заморачивался

runAllMigrations :: IO ()
runAllMigrations = do
conf <- C.retrieveConfig
conf <- C.loadConfig "./config/dev.dhall"
Copy link
Contributor

Choose a reason for hiding this comment

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

это точно надо поправить, нехорошо что везде путь таскать надо. В файле конфига его константой сохранить хотя бы

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Это я оставил в туду в описании PR. Если у кого будет возможность то можно исправить.

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.

4 participants