Migration à Rails 6

[n-b]

• Attendre que les gems GraphQL soient compatible Rails 6
• Utiliser des versions upstream des gems (plutôt que nos forks)
• Mettre à jour Capybara (Bumper Capybara #4536)
• Mettre à jour les gems
• Faire passer les tests avec Rails 6
• Tester l'application manuellement
• Déployer le passage à Rails 6

Après la migration

• Passer de la config par défaut Rails 5 à Rails 6
• Activer les belongs_to requis par défaut
• Refactorer le chargement de BlobVirusScannerConcern (cf. initializers/active_storage.rb)
• Ré-activer le linter Rails/UniqueValidationWithoutIndex, et rajouter des index aux colonnes concernées (Ajouter des index aux colonnes uniques #6013)
• Activer zeitwerk

[kemenaran]

Pour l'instant je crois qu'on a quelques points bloquants :

• La compatibilité des gems GraphQL avec Rails 6
• Un peu de code dans une gem forkée à nous, qu'il faudrait qu'on nettoie.

@tchak a déjà pas mal travaillé dessus ; je pense que c'est bien s'il termine le sujet lui-même. (/cc @pengfeidong)

[kemenaran]

@tchak j'ai essayé de bumper les gems (hors passage à Rails 6). Mais je tombe sur la mise à jour de Sprockets qui nous demande de créer un fichier app/assets/config/manifest.js. C'est pas quelque chose que tu aurais déjà fait dans une branche, par hasard ?

[tchak]

@kemenaran j'ai bloqué la version de sprockets – migrer pose des problèmes, ce n’est pas nécessaire pour rails 6 et on s'en sert de moins en moi de sprockets. J'ai bon espoir qu'on vire sprockets entièrement un de ces jours.

[n-b]

Le manifest.js remplace simplement la ligne Rails.application.config.assets.precompile += ... dans initializers/assets.rb. La doc est plutôt bien fichue. On a eu un souci sur PdE, parce qu’on avait pas retiré la ligne dans assets.rb, que certains fichiers étaient compilés plusieurs fois, que ça se terminait en race condition qui faisait crasher sassc.

[kemenaran]

État des lieux : y'a une branche, qui marche, mais sur laquelle il faut corriger des tests liés aux PJ

[kemenaran]

On est maintenant sur Rails 6 !

Maintenant, reste le nettoyage post-migration. (Voir la liste ci-dessus.)

[kemenaran]

Pour zeitwerk, ça peut s'activer en 1/4h de travail à peine : y'a une juste tâche rake zeitwerk:check qui dit de renommer quelques constantes, on fait ça, et hop.

Le seul souci, c'est que le debugger qu'on utilise, byebug, n'est pas compatible avec zeitwerk (deivid-rodriguez/byebug#564). On peut donc soit:

• Changer de débogueur (y'en a des compatibles)
• Attendre un moment
• Contribuer une compat avec zeitwerk

Dans tous les cas, y'a pas d'urgence. On peut remettre ça à plus tard.

[kemenaran]

J'ai l'impression que globalement la migration est faite.

Je crée une issue pour le seul truc qui restait, et hop.