Produits Entr'ouvert » Combo

J'allais dire que ça flingue le support de sqlite mais Jenkins l'a dit avant moi.

(j'étais à penser qu'on avait déjà comme ailleurs viré ça de tox et cie).

Frédéric Péters a écrit :

(j'étais à penser qu'on avait déjà comme ailleurs viré ça de tox et cie).

Alors c'est amusant mais je pensais que tu étais celui qui tenait à garder sqlite pour Combo (panik ou autre je ne sais plus d'où ça vient...). Mais si on peut le faire sauter et gagner un round dans les tests, et ce "vrai" jsonfield, alors c'est parfait.

Lauréline, penses-tu qu'on pourrait inclure dans les tests "quelque chose" qui montre que cette migration fonctionne effectivement ? Ou bien on se base juste sur le faire que "makemigrations" ne dit rien. (vraie question)

makemigrations ne dit rien :)
testé en local c'est ok; enfin, on assume qu'on a bien déjà des colonnes jsonb partout

donc on enlève le support sqlite ?

Alors c'est amusant mais je pensais que tu étais celui qui tenait à garder sqlite pour Combo (panik ou autre je ne sais plus d'où ça vient...).

Yes, mais bon...

Oui on retire la prise en charge sqlite.

Lauréline Guerin a écrit :

on assume qu'on a bien déjà des colonnes jsonb partout

Dans passerelle ce n'était pas le cas et ça avait occasionné moult tickets (#43492, entre autres). Je ne sais pas d'où vient l'assurance que c'est bon ici ?

OK c'est géré à part dans #52915.