etalab / rncs_worker_api_entreprise

API pour récupérer et mettre à disposition les données du Répertoire National du Commerce et des Société
https://entreprise.data.gouv.fr
MIT License
23 stars 13 forks source link

Upgrade to Rails 6 #138

Closed brindu closed 5 years ago

brindu commented 5 years ago

Contenu de la PR

Migration vers Rails 6.0 ! C'est la première de tous nos différents répo de sources alors je vous invite à review sérieusement 😄

J'en ai profité pour supprimer quelques dépendances, notamment activerecord-import car l'insert de masse est maintenant disponible au travers d'ActiveRecord.

L'autoload est géré par une nouvelle gem, ça a pas l'air mal car ça m'a forcé à être consistent sur le naming des acronymes (on avait par exemple soit des "Tc" et "TC", au choix, qui ne passaient plus l'autoloading et j'ai dû mettre des "TC" partout).

La gestion de la base de données dans l'environnement de tests a elle aussi changé : j'ai du donner le privilège SUPERUSER à l'utilisateur PostgreSQL car lancer les tests induit maintenant une suppression de la BDD, une création, puis le run des migrations. Et les migrations échouent (à cause de l'extension "pgcrypto") sans ce privilège. Rien de très grave puisqu'il s'agit de l'utilisateur en environnement de dev et de test.

Comment faire la review

Commits par commits ; j'ai souvent explicité le "pourquoi" des changements en commentaires dans les commits, n'hésitez pas à y jeter un oeil.

Closes #114