Refactoring

[bonjourmauko]

TL; TR :

• [x] Opinionated ruby styling (pour inviter la collaboration, cf ruby style guide).
• [x] Module function et mixin pattern pour ne pas polluer le scope global en Ruby (cf here).
• [x] Plus de « readability » (en « trade-off » d'un peu des LOCs, cf here et Fowler et Beck).
• [x] Séparation des couches de service : API/Notifications (cf Alistair).
• [x] CI configuration (Rakefile + Travis).
• [x] Assurer l'encodage en utf-8.
• [x] Réduction d'« object allocation memory »

  Pollution du scope global

La gem sendgrid-ruby utilise à son tour la gem ruby-http-client laquelle, malheureusement, utilise un « catch-all » pour les méthodes non définis :

# Dynamically add segments to the url, then call a method.
# (e.g. client.name.name.get())
#
# * *Args*    :
#   - The args are autmoatically passed in
# * *Returns* :
#   - Client object or Response object
#
def method_missing(name, *args, &_block)
  # Capture the version
  if name.to_s == 'version'
    @version = args[0]
    return _
  end
  # We have reached the end of the method chain, make the API call
  return build_request(name, args) if @methods.include?(name.to_s)
  # Add a segment to the URL
  _(name)
end

Cela dit, donc, include SendGrid dans le scope global de l'application ne peut que générer des effets imprévisibles.

[bonjourmauko]

fixup + rebase

[bonjourmauko]

@Morendil Si cela te paraît plus convenable, je peux éclater le PR en plusieurs de manière à qu'ils soient plus atomiques.

[Morendil]

@maukoquiroga En fait j'aurais surtout besoin qu'on puisse binômer 1h pour comprendre tes choix, et on a peu eu l'occasion ces dernières semaines... J'ai bon espoir que ça se fasse bientôt :)

[Morendil]

Closed via #3