Да, тут согласен. вещей может быть и больше гораздо.

и в правильном случает каждый из методов типа insert_or_ignore, add_to_map лучше бы дергал внешний чей то интерфейс внутри себя

Совсем недавно был у меня кейс, где нужно протестировать метод, который просто состоит из цепочки пайпов к внешним методам. Я так и не смог понять что тестировать в нем. решил не делать

Функции в цепочке пайпов по сути либо являются обертками к нормальному api, получается такая себе велосипедная композиция, либо это builder паттерн - как например в changeset или multi

Шанс что у тебя такая красивая цепочка пайпов и ты не просто понторез, который вчера узнал про этот синтаксис и везде его суёшь стремится к нулю

цепочка пайпов дает хорошую читаемость, в ущерб(в рассмотренном случае) правильному интерфейсу функций.

@Virviil это можно закрепить в шапке вместо текущей неактуальной вакансии

children = 
  [
    MyApp.Repo,
    MyApp.Endpoint
  ] ++
    case Mix.env() do
      :test -> []
      _ -> [MyApp.Scheduler]
    end

Такой код не будет работать с релизами (в рантайме), т.к. там Mix не доступен обычно (если конечно этот код не выполняется при компиляции модуля)

А предложенный тобой - будет? Там тоже используется Mix.env

Будет, так как там Mix дёргается во время компиляции, а не во время выполнения

У тебя просто в зависимости от окружения после компиляции будут разные функции env_specific_children

ага, понял разницу

Работа только офис, удаленку не рассматриваете?

Офис))

Потом возможно, частично

Большое спасибо за советы, ребята.
Действительно, нужно было больше бэкграунда от меня в самом начале.
Повторюсь, что фп и elixir для меня вновинку, не больше двух недель с ними ознакамливаюсь.
Если более глобально по задаче, то алгоритм сервиса примерно следующий: parse_data |> get_or_create_city |> insert_data |> get_another_info_from_api.
Делал логику через pipeline для удобочитаемости, но это добавило задачу обработки ошибок. try/rescue не рассматривал, т.к. помещать всю логику в один большой блок try - моветон. Поэтому информацию передаваемую между функциями завернул в {:ok | :error, %{}}. Пришлось, конечно, используя pattern matching, добавлять конструкции типа foo({:error, _error}) для всех функций цепочки, но другого метода тогда я не придумал. Сейчас цепочку разорвал и таких конструкций в коде поубавилось, но они всё равно есть.
Касательно самого вопроса по созданию city. Да, в конечно итоге пришёл к get_or_create_city, код на скрине.

👍

если на инсерт ошибка будет (unique constraint, например) как обрабатывается? (или неактуально?)

Use with, Luke!