Okay the thing is - I plan to add block support for D_ methods upstream, then we can remove duplicated code in Foreman.

Please do not put this on Sprint until let's say 20, I will need to work upstream first.

I wonder if this is worth the effort, these are three methods on three lines, it's only used in gettext debug (mark_translated). Upstream, passing block can cause issues because the multi domain version needs to call it twice - could cause side effects and comparsion could we wrong for some values (e.g. new instance returned).

    def D_(key)
      FastGettext.translation_repositories.each_key do |domain|
        result = FastGettext::TranslationMultidomain.d_(domain, key) {nil}
        return result unless result.nil?
      end
      key
    end

In the current upstream code safe value (nil) is used. I am closing this one.