split rendering in more maintainable files

[jdeniau]

Pull request :twisted_rightwards_arrows: created by @RomainMapado on 2017-01-27 15:02 Last updated on 2017-01-30 08:33 Original Bitbucket pull request id: 46

Participants:

• @jdeniau (reviewer) :heavy_check_mark:
• @jnieuviarts (reviewer)
• @zwit (reviewer) :heavy_check_mark:
• @dallegoet (reviewer) :heavy_check_mark:

Source: https://github.com/mapado/datection/commit/bc9d4c66813f on branch rs-feat-split-rendering Destination: https://github.com/mapado/datection/commit/54a88baac0c8 on branch master Merge commit: https://github.com/mapado/datection/commit/eaed28fe02be

State: MERGED

C'est moins violent que ca en a l'air:

• Split du render.py en fichiers plus petits (dans rendering/) : ca sera plus maintenable quand on va ajouter les logiques pour de nouvelles langues. render.py garde les points d'entrée (display() et get_display_schedule()) et la logique de choix du meilleur rendering.

• Refacto du Long formatter en FullFormatter=LongFormatter+ExclusionFormatter (on avait des dépendances cycliques).

• La locale est maintenant un attribut de classe des formatters et non plus une variable globale changée à la volée.

[jdeniau]

@jdeniau approved :heavy_check_mark: the pull request on 2017-01-27 15:27

[jdeniau]

@jdeniau commented on 2017-01-27 15:27

J'approve la description ;)

[jdeniau]

@jdeniau commented on 2017-01-27 15:28

J'approve la description ;)

Probablement a tagguer en majeur vu le changement de nom etc.

[jdeniau]

@zwit commented on 2017-01-27 15:32

Je crois que je vais faire pareil x)

[jdeniau]

@zwit approved :heavy_check_mark: the pull request on 2017-01-27 15:32

[jdeniau]

@dallegoet approved :heavy_check_mark: the pull request on 2017-01-27 16:23