Понятное дело

Есть претензии - можешь смело высказать их Матцу)

Кстати, можно просто пропатчить класс, делается легко. Будет тебе хоть строка

Что то не могу понять чего рубокоп ругается на это: if spec.respond_to?(:metadata) spec.metadata["allowed_push_host"] = "https://rubygems.org" else raise "RubyGems 2.0 or newer is required to protect against public gem pushes." end Use a guard clause instead of wrapping the code inside a conditional expression. (Style/GuardClause)

в общем, убери else и поставь на его место end

в общем, убери else и поставь на его место end

нет

видимо потому что метод состоит только из этого условия

Что то не могу понять чего рубокоп ругается на это: if spec.respond_to?(:metadata) spec.metadata["allowed_push_host"] = "https://rubygems.org" else raise "RubyGems 2.0 or newer is required to protect against public gem pushes." end Use a guard clause instead of wrapping the code inside a conditional expression. (Style/GuardClause)

почитай сам лучше. «граничное условие» называется

сделай raise unless

а остальное без условия

сделай raise unless

?

видимо потому что метод состоит только из этого условия

не обязательно только, достаточно чтобы некое условие стояло в конце (даже без else)

Стояло в конце? ты имеешь ввиду blablah unless и т.д? я знаю что такое guard clause. Я не понимаю чего он именно к этому куску прицепился.

Выглядит вполне логичным условием.

Хотя возможно именнно потому что он это понял и решил что логичнее просто raise сначала ввести вместо такого условия.

зато если сделаешь raise unless, то будет ругаться на длину строки

я бы оставил так

Стояло в конце? ты имеешь ввиду blablah unless и т.д? я знаю что такое guard clause. Я не понимаю чего он именно к этому куску прицепился.

def method # some action # some another action if condition # action for condition end end необязательно, чтобы метод из условия состоял. здесь важное, что условие в конце метода, перед результатом

Я понял тебя, спасибо.

Хотя возможно именнно потому что он это понял и решил что логичнее просто raise сначала ввести вместо такого условия.

смысл всего этого — чтобы тело метода (его логика) была без лишней индентации, а невалидные случаи отсекались проверкой раньше

unless spec.respond_to?(:metadata) raise "RubyGems 2.0 or newer is required to protect against public gem pushes." end spec.metadata["allowed_push_host"] = "https://rubygems.org"

Так норм вполне.

Так норм вполне.

да, вполне. строчку можно вынести в переводы или константу, например, но да.

Это gemspec

этот код по дефолту бандлером генерится.

https://github.com/Fedcomp/active_sms А вот кстати сам гем. Унифицированная отправка смсок в руби

допилил таки

>active ну такоэ

Ага, теперь хоть юзать можно.

>active ну такоэ

Ну да.

https://github.com/Fedcomp/active_sms А вот кстати сам гем. Унифицированная отправка смсок в руби

молодец. но на мой вкус функциональщина (классовые методы). а ещё неясно, например, может ли работать с mcommmunicator, например

>active ну такоэ

да, тоже смутило, но спасибо, что без active_support в зависимостях

Гем сам по сбее ни с кем не работает

да, тоже смутило, но спасибо, что без active_support в зависимостях

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

Гем сам по сбее ни с кем не работает

да я понял, но не вижу нормального примера конфига / написания прослойки

Теперь поверх него будет версия для рельс, но потом.

да я понял, но не вижу нормального примера конфига / написания прослойки

Есть в доках пример написания своего обработчика.

https://github.com/Fedcomp/active_sms#writing-your-own-sms-backend

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

не так уж и сложно) я разве что себе иногда помогаю своим же gem'ом альтернативой active_support (с другим механизмом работы), есть замены функциональные, а так несложно

ну так легче взять гем компании https://github.com/asiniy/sms_ru

и отправлять сразу без ебли

а тут бекенд какой-то писать

ну так легче взять гем компании https://github.com/asiniy/sms_ru

фишка в том, что не только sms_ru позволяет отправлять смс

фишка в том, что не только sms_ru позволяет отправлять смс

фишка в том, что ты обычно юзаешь апи какой-то компании для отправки смс

не кучу, а одну

а абстракции — это хорошо. у тебя один интерфейс, замена кода минимальна. но пригождается редко, да

не кучу, а одну

здесь на руку прослойка, если провайдер сменится

фишка в том, что ты обычно юзаешь апи какой-то компании для отправки смс

А теперь ты хочешь на локалке не отправлять смс

А теперь ты хочешь на локалке не отправлять смс

почему не отправлять?

Но тем не менее видеть код

что ты делаешь?

потому что на локалке у тебя фронтендер может разрабатывать например

с какой стати он будет реальные смски слать?

я сделал себе воркер который смотрит ивенты в amqp и отправляет нотификации

почему не отправлять?

ну потому что зах для тестов одной вещи тянуть ещё отправку смс? ну, деньги тратить на это

а теперь ты понял что smsru гавно, что теперь? переписывать весь код который его юзает.

Проще взять унифицированный интерфейс с кучей реализаций.

Особенно если у тебя нет предпочтения по сервису.

а теперь ты понял что smsru гавно, что теперь? переписывать весь код который его юзает.

ну в моем случае это один воркер на все

у нас в разное время юзалось 4 разных провайдера для смс

Проще взять унифицированный интерфейс с кучей реализаций.

вот только их не хватает (для примера хотя бы)

то дорого, то даунтайм в два дня у провайдера, то еще что-то

Да, согласен. Это тоже важная часть

у нас в разное время юзалось 4 разных провайдера для смс

вот да, у меня такого нет, но знаю, что бывает

в дальнейшем напилю побольшще реализаций.

то дорого, то даунтайм в два дня у провайдера, то еще что-то

Тоже верно. Можно относительно быстро перепрыгнуть на другого провайдера.

ERROR: S client not available

так. зачем юзают active_record и sequel, если есть pg и mysql? ну, эээм

(не очень корректный пример, но утрированный и доходчивый, я надеюсь)

Тоже верно. Можно относительно быстро перепрыгнуть на другого провайдера.

дак тебе надо бекенд написать в твоем случае для начала

нихуя себе быстро

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

дак тебе надо бекенд написать в твоем случае для начала

У меня будет некоторое количество готовых реализаций.

нихуя себе быстро

это может занять пол-дня, не 1-2

смс-бэкенд на самом деле быстро пишется :/

это может занять пол-дня, не 1-2

ты можешь взять готовый гем компании (который обычно есть)

Который vendor lock.

и что лол

потому что меняешь один (!) файл, а не всё, где делается отправка смс. если ты молодец — то ты уже написал свою прослойку для этого. если нет — вот гем. полезная вещь

switch case

ты можешь взять готовый гем компании (который обычно есть)

и менять код по всему проекту, ага. весело

и менять код по всему проекту, ага. весело

воркер для нотификаций

а можешь ЕГО ЖЕ (вендроный гем) заюзать на бэкенде прослойки

воркер для нотификаций

не все любят воркеры. советую поискать статью на тему, почему подход с воркерами плохой.

воркер для нотификаций

Смски могут не только в воркере юзаться

разные юзкейсы бывают.

не спорю, сам совал в воркера и везде его юзал.

Смски могут не только в воркере юзаться

что значит не только в воркере юзаться?

Но в рубишном скрипте например воркеров не будет.

у тебя в amqp заходит заявка

типа отправь тому-то

воркер смотрит, отправляет

у тебя в amqp заходит заявка

у многих amqp?

Это в рельсах.

У меня общерубишный гем.

каких рельсах бля

берешь ставишь rabbitmq докером

или не докером

и сидишь слушаешь отдельным рубишным скриптом что там происходит