@return mixed

return json_decode($response->getBody());

ну он учится, ему нужно постараться расписать почему

Чтобы в будущем могли использовать вместо Guzzle чето другое, implements мой Interface?

тип того

ну и если конфиг уже, то не нужно юзать env()

лучше енв в коде помимо конфигов поменьше юзать

да в целом не стоит обращаться из клиентского кода к config/env

ну и в целом от подобного вроде же нет толку )
      try {
           return json_decode($response->getBody());
       } catch (Exception $exception) {
           throw new Exception($exception->getMessage(), $exception->getCode());
       }

@return mixed

return json_decode($response->getBody());

про дто не думал, спасибо

можно залогировать дополнительно

ну да, но логов там нет)

о, это тестовое

@return mixed

return json_decode($response->getBody());

0: cs fixer понятно наверное
1: protected не нужны потому что наследование не нужно
2: Ну типы надо ставить
3: Ой, это ж ещё и не симфа наверн. Вобщем хедерами в идеале пусть хттп клиент сам рулит, и они задаются где-то в конфиге, но это ещё +- норм, ок
4:  Ну return mixed ни о чем, нужна DTO
5: getWeather, хотя бы, я бы и сам класс переназвал бы как-нибудь без суффикса service, мол YandexWeatherClient
6: Нужна DTO с проставленными типами, и маппинг результата на неё. Хоть ручками, хоть через сериалайзер какой, чтобы во первых было видно что возвращается и как с этим работать, во вторых проверялось хотя-бы в рантайме. Сейчас неясно что возвращается + будет боль если апиху сломают, потому что упадёт в коде даже не клиент к апихе, а какой-то класс работающий с респонсом
7: Ну из конфигов надо либ собрать структурку(DTO) и прокинуть в конструктор, либо просто в классе захардкодить, это ж параметры апихи, они не будут переиспользоваься

+
еще тут кучу всего крутого
https://github.com/Hexlet/ru-test-assignments

Постарался добавить пояснений

норм

@forszaken давай удаляй и по новой с учетом замечаний

спасибо!

это же не реальное тестовое?

ну да, сейчас переделаю и закомитю

нет