[12.0] [MIG] mail_via

[chafique-delli]

No description provided.

[bealdav]

incoming

[florian-dacosta]

@chafique-delli Peut être ajouter OCA dans les auteurs pour que travis soit content ?
Dans un sens, le but est éventuellement que ces modules soient proposés à l'OCA, donc ca ne me choque pas.
Ou alors enlever ce check...
@bealdav un avis?

[rvalyi]

Dans Shopinvader y'a les tests via le maintainer-tools sans l'OCA dans les authors ( https://github.com/shopinvader/odoo-pim/blob/12.0/pim/__manifest__.py#L8 ) , après j'ai pas encore pigé comment le test y est désactivé.

[rvalyi]

ouai du coup ça semble être a cause du LINT_CHECK="0" ici https://github.com/shopinvader/odoo-pim/blob/12.0/.travis.yml#L31
que le teste est desactivé. Alors que dans le meme temps le test du pre-commit doit qd meme verifier une partie des règles. Bon sur ak-odoo-incubator, l'OCA dans les author c'est pas choquant non plus.

[bealdav]

A cat is a cat, not other thing. If not in OCA then OCA must NOT be author.

But this module should go in OCA to be maintained by others and we need it for v14. It's not complex and is generic

my 2cts

[bealdav]

Florian, je crois que ce check est enlevé en v10 au moins dans la config

Je pense que c ça qui fait le boulot

https://github.com/akretion/ak-odoo-incubator/blob/10.0/.travis.yml#L41-L42

Il faut adapter la config v12 par rapport à la v10 du coup

[codecov]

Codecov Report

Merging #198 into 12.0 will decrease coverage by 2.22%.
The diff coverage is 45.00%.

@@            Coverage Diff             @@
##             12.0     #198      +/-   ##
==========================================
- Coverage   51.35%   49.12%   -2.23%     
==========================================
  Files           2        4       +2     
  Lines          37       57      +20     
==========================================
+ Hits           19       28       +9     
- Misses         18       29      +11     

Impacted Files	Coverage Δ
mail_via/models/ir_mail_server.py	40.00% <40.00%> (ø)
mail_via/models/mail_mail.py	50.00% <50.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38aaad9...87ab208. Read the comment docs.